-
-
Notifications
You must be signed in to change notification settings - Fork 0
chore: R-16 toolchain pin + reconciliation #76
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
281e68c
d80aa61
a982cce
6445951
940bf90
450c443
30da6eb
ef436f8
ae1207b
e6572fe
d54d317
b8ba659
5c898c7
333c967
85be3b3
026923d
0c614a0
4c76b41
fddde3e
7ece6dd
01c972e
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,2 @@ | ||
| [tools] | ||
| just = "1.36.0" |
This file was deleted.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,47 @@ | ||
| # Architecture | ||
|
|
||
| ## Overview | ||
|
|
||
| This repository follows a modular, maintainable architecture designed for clarity, scalability, and long-term sustainability. | ||
|
|
||
| ## Directory Structure | ||
|
|
||
| ``` | ||
| . | ||
| ├── src/ # Source code | ||
| ├── tests/ # Test suites | ||
| ├── docs/ # Documentation | ||
| ├── scripts/ # Utility scripts | ||
| ├── config/ # Configuration files | ||
| ├── LICENSE # License file | ||
| ├── LICENSES/ # Full license texts | ||
| └── README.adoc # Project documentation | ||
|
Comment on lines
+11
to
+18
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win Keep the directory map consistent with
🤖 Prompt for AI Agents |
||
| ``` | ||
|
|
||
| ## Design Principles | ||
|
|
||
| - **Separation of Concerns**: Each module has a single responsibility | ||
| - **Testability**: Code is written to be easily testable | ||
| - **Documentation**: All public APIs are documented | ||
| - **Configuration**: Environment-specific settings are externalized | ||
|
|
||
| ## Dependencies | ||
|
|
||
| - External dependencies are minimized and clearly declared | ||
| - Version pinning is used for reproducibility | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win Qualify the version-pinning claim.
🤖 Prompt for AI Agents |
||
|
|
||
| ## Security Considerations | ||
|
|
||
| - Sensitive data is never committed to the repository | ||
| - Secrets are managed through environment variables or secure vaults | ||
| - Regular dependency audits are performed | ||
|
|
||
| ## Maintainability | ||
|
|
||
| - Code follows consistent style guidelines | ||
| - Pull requests require review and CI checks | ||
| - Issues and discussions are tracked transparently | ||
|
|
||
| --- | ||
|
|
||
| *Last updated: 2026-07-18* | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,60 @@ | ||
| # Governance | ||
|
|
||
| ## Overview | ||
|
|
||
| This project is governed by the following principles and structures to ensure transparent, inclusive, and effective decision-making. | ||
|
|
||
| ## Roles and Responsibilities | ||
|
|
||
| ### Maintainers | ||
|
|
||
| Maintainers are responsible for: | ||
| - Reviewing and merging pull requests | ||
| - Managing releases and versioning | ||
| - Ensuring code quality and standards | ||
| - Triaging issues and bug reports | ||
| - Community engagement and support | ||
|
|
||
| ### Contributors | ||
|
|
||
| Contributors are expected to: | ||
| - Follow the code of conduct | ||
| - Submit well-documented pull requests | ||
| - Write tests for new functionality | ||
| - Maintain existing tests | ||
| - Update documentation as needed | ||
|
|
||
| ## Decision Making | ||
|
|
||
| ### Minor Changes | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 📐 Maintainability & Code Quality | 🔵 Trivial | ⚡ Quick win Add blank lines after the subsection headings.
Also applies to: 33-33, 38-38 🧰 Tools🪛 markdownlint-cli2 (0.23.2)[warning] 29-29: Headings should be surrounded by blank lines (MD022, blanks-around-headings) 🤖 Prompt for AI AgentsSource: Linters/SAST tools |
||
| - Can be made by any maintainer | ||
| - Include bug fixes, documentation updates, dependency updates | ||
|
|
||
| ### Major Changes | ||
| - Require discussion in issues or pull requests | ||
| - Include new features, architectural changes, API changes | ||
| - Need approval from at least 2 maintainers | ||
|
|
||
| ### Breaking Changes | ||
| - Require RFC (Request for Comments) process | ||
| - Need approval from majority of maintainers | ||
|
Comment on lines
+36
to
+40
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win 🔎 Supported by static analysis🏁 Script executed: #!/bin/bash
printf '%s\n' '--- GOVERNANCE.md ---'
sed -n '1,90p' GOVERNANCE.md
printf '%s\n' '--- MAINTAINERS.adoc ---'
sed -n '1,140p' MAINTAINERS.adocRepository: hyperpolymath/empty-linter Length of output: 3152 Reconcile the approval rules with
🧰 Tools🪛 LanguageTool[grammar] ~40-~40: An article may be missing. (BUNCH_OF) 🪛 markdownlint-cli2 (0.23.2)[warning] 38-38: Headings should be surrounded by blank lines (MD022, blanks-around-headings) 🤖 Prompt for AI Agents |
||
| - Must include migration guide | ||
|
|
||
| ## Code of Conduct | ||
|
|
||
| All participants are expected to follow our Code of Conduct. Violations can be reported to the maintainers. | ||
|
|
||
| ## Communication | ||
|
|
||
| - **Issues**: For bug reports and feature requests | ||
| - **Discussions**: For questions and general discussion | ||
| - **Pull Requests**: For code contributions | ||
|
|
||
| ## Licensing | ||
|
|
||
| All contributions are made under the terms of the repository's LICENSE file. | ||
| By submitting a pull request, you agree to license your contributions accordingly. | ||
|
Comment on lines
+55
to
+56
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🔒 Security & Privacy | 🟡 Minor | ⚡ Quick win 🔎 Supported by static analysis🏁 Script executed: #!/bin/bash
set -eu
printf '%s\n' '--- GOVERNANCE.md ---'
cat -n GOVERNANCE.md | sed -n '1,90p'
printf '%s\n' '--- candidate policy files ---'
git ls-files | grep -E '(^|/)(LICENSE|LICENSE\.[^/]+|CONTRIBUTING[^/]*|MAINTAINERS[^/]*|CODE_OF_CONDUCT[^/]*)$' || true
printf '%s\n' '--- contribution/agreement references ---'
rg -n -i 'CLA|DCO|developer certificate|sign[- ]off|license your contributions|licen[cs].*contribut|contribut.*licen[cs]|pull request|pull-request|agreement' \
GOVERNANCE.md LICENSE* CONTRIBUTING* MAINTAINERS* CODE_OF_CONDUCT* .github 2>/dev/null || trueRepository: hyperpolymath/empty-linter Length of output: 12617 🏁 Script executed: #!/bin/bash
set -eu
printf '%s\n' '--- LICENSE: identity and contribution terms ---'
cat -n LICENSE | sed -n '1,175p'
printf '%s\n' '--- LICENSE: agreement and closing terms ---'
cat -n LICENSE | sed -n '300,330p'
printf '%s\n' '--- CONTRIBUTING.adoc ---'
cat -n CONTRIBUTING.adocRepository: hyperpolymath/empty-linter Length of output: 13365 Align the contribution licence statement with
🧰 Tools🪛 LanguageTool[locale-violation] ~55-~55: LICENSE must be spelled with a “c” when used as a noun in British English. Use “licence”. (LICENCE_LICENSE_NOUN_SINGULAR) 🤖 Prompt for AI Agents |
||
|
|
||
| --- | ||
|
|
||
| *Last updated: 2026-07-18* | ||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win
Add a language identifier to the directory-tree fence.
Use
textafter the opening fence. This resolves markdownlint MD040.🧰 Tools
🪛 markdownlint-cli2 (0.23.2)
[warning] 9-9: Fenced code blocks should have a language specified
(MD040, fenced-code-language)
🤖 Prompt for AI Agents
Source: Linters/SAST tools