Repository navigation
docs(brand): write the product name as Beatrax - #3
Merged
Merged
Conversation
The name is being restyled with a capital B. This updates every user-facing surface the site owns — the nav and footer wordmarks, the page titles and structured-data name (through site.name), the English and Dutch copy, the feature cards, the comparison tables, and the served llms.txt / robots.txt — to write "Beatrax" in prose. Left lower-case on purpose: URLs and identifiers (beatrax.app, the beatrax-app org, the beatrax repo name), the CLI commands and release-asset filenames in the reproduced changelog, and the English FAQ questions and release bodies, which are fetched at build time from the spec and the GitHub API and read lower-case there. The faqNl lookup keys stay lower-case so they keep matching the English questions the spec still serves, while their Dutch values adopt the capital. Depends on beatrax-app/spec#9, which flips DES-R1 to require the capital. Spec: DES-R1 Signed-off-by: Wessel Verheij <info@nightworks.io>
github-actions
Bot
added
dependencies
📎 Spec referencesThis PR cites the following spec identifiers:
Updated automatically on each push. |
The dependency scanner is a required check and reports three high-severity transitive advisories against the committed lockfile: fast-uri 3.1.4 (GHSA-7p8r-x3mc-p8w7), js-yaml 4.3.0 (GHSA-5p4m-2wfm-xmqj) and nanoid 3.3.16 (GHSA-2v37-7h3g-55p8). All three fixes are patch releases inside the existing semver ranges, so this is a lockfile-only bump — package.json is untouched. `npm audit` now reports zero vulnerabilities and the site still type-checks and builds all 33 pages. Spec: GOV-R12 Signed-off-by: Wessel Verheij <info@nightworks.io>
Automatic analysis flags the i18n files as ~99% duplicated, which fails the quality gate's new-code duplication condition. The duplication is by design: en.ts / nl.ts and pages-en.ts / pages-nl.ts are parallel translations of one structure, defined key-for-key so the type system stops the two locales from drifting. Copy-paste detection has nothing useful to say about that, so the locale files are excluded from CPD only — bug, vulnerability and maintainability analysis still runs on them. Automatic analysis ignores sonar-project.properties, so the setting lives in .sonarcloud.properties, the file it does read. Spec: GOV-R12 Signed-off-by: Wessel Verheij <info@nightworks.io>
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Writes the product name with a capital B — Beatrax — across every user-facing surface the site owns:
<title>, JSON-LDname, andog:site_name(viasite.name)en.ts,pages-en.ts), Dutch copy (nl.ts,pages-nl.ts), feature cards, comparison tablespublic/llms.txtandpublic/robots.txtpackage.jsondescriptionDeliberately left lower-case
beatrax.app, thebeatrax-apporg, thebeatraxrepo namephp artisan beatrax:setup) and release-asset filenames (beatrax_1.3.0_amd64.deb) in the reproduced changelog seedfaq.md) and the GitHub API, which still serve lower-case. Capitalising those belongs in the spec/app repos, not here.faqNllookup keys stay lower-case so they keep matching the English questions the spec serves; only their Dutch values adopt the capital.Verification
npm run check→ 0 errors / 0 warnings (69 files)npm run build→ 33 pages built; Dutch FAQ confirmed rendering translated "Beatrax" (not English fallback), title/og confirmed "Beatrax"Dependency
Depends on beatrax-app/spec#9, which flips DES-R1 to require the capital. That should merge first (GOV-R4 ordering). On merge here,
deploy.ymlpublishes to GitHub Pages.Spec: DES-R1