-
-
Notifications
You must be signed in to change notification settings - Fork 0
fix(docs): repair the contributing guide content defects #61
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
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 |
|---|---|---|
| @@ -1,44 +1,59 @@ | ||
| # Clone the repository | ||
|
|
||
| git clone <https://github.com/hyperpolymath/nextgen-languages.git> cd | ||
| nextgen-languages | ||
| git clone https://github.com/hyperpolymath/nextgen-languages.git | ||
| cd error-lang | ||
|
coderabbitai[bot] marked this conversation as resolved.
|
||
|
|
||
| # Using Guix (recommended for reproducibility) | ||
|
|
||
| guix develop | ||
|
|
||
| # Or using toolbox/distrobox | ||
|
|
||
| toolbox create nextgen-languages-dev toolbox enter nextgen-languages-dev | ||
| \# Install dependencies manually | ||
| toolbox create nextgen-languages-dev | ||
| toolbox enter nextgen-languages-dev | ||
| # Install dependencies manually | ||
|
|
||
| # Verify setup | ||
|
|
||
| just check \# or: cargo check / mix compile / etc. just test \# Run test | ||
| suite | ||
|
|
||
|
|
||
| ### Repository Structure | ||
|
|
||
| nextgen-languages/ ├── src/ \# Source code (Perimeter 1-2) ├── lib/ \# | ||
| Library code (Perimeter 1-2) ├── extensions/ \# Extensions (Perimeter 2) | ||
| ├── plugins/ \# Plugins (Perimeter 2) ├── tools/ \# Tooling (Perimeter | ||
| 2) ├── docs/ \# Documentation (Perimeter 3) │ ├── architecture/ \# ADRs, | ||
| specs (Perimeter 2) │ └── proposals/ \# RFCs (Perimeter 3) ├── examples/ | ||
| \# Examples (Perimeter 3) ├── spec/ \# Spec tests (Perimeter 3) ├── | ||
| tests/ \# Test suite (Perimeter 2-3) ├── .well-known/ \# Protocol files | ||
| (Perimeter 1-3) ├── .github/ \# GitHub config (Perimeter 1) │ ├── | ||
| ISSUE_TEMPLATE/ │ └── workflows/ ├── CHANGELOG.md ├── CODE_OF_CONDUCT.md | ||
| ├── CONTRIBUTING.md \# This file ├── GOVERNANCE.md ├── LICENSE ├── | ||
| MAINTAINERS.md ├── README.adoc ├── SECURITY.md ├── flake.guix \# Guix | ||
| flake (Perimeter 1) └── Justfile \# Task runner (Perimeter 1) | ||
|
|
||
| just doctor # Check environment | ||
| just test # Run test suite | ||
|
|
||
| ### Repository Structure | ||
|
|
||
| ```text | ||
| error-lang/ | ||
| ├── src/ # Source code (Perimeter 1-2) | ||
| ├── lib/ # Library code (Perimeter 1-2) | ||
| ├── extensions/ # Extensions (Perimeter 2) | ||
| ├── plugins/ # Plugins (Perimeter 2) | ||
| ├── tools/ # Tooling (Perimeter 2) | ||
| ├── docs/ # Documentation (Perimeter 3) | ||
| │ ├── architecture/ # ADRs, specs (Perimeter 2) | ||
| │ └── proposals/ # RFCs (Perimeter 3) | ||
| ├── examples/ # Examples (Perimeter 3) | ||
| ├── spec/ # Spec tests (Perimeter 3) | ||
| ├── tests/ # Test suite (Perimeter 2-3) | ||
| ├── .well-known/ # Protocol files (Perimeter 1-3) | ||
| ├── .github/ # GitHub config (Perimeter 1) | ||
| │ ├── CONTRIBUTING.md # This file | ||
| │ ├── ISSUE_TEMPLATE/ | ||
| │ └── workflows/ | ||
| ├── CHANGELOG.md | ||
| ├── CODE_OF_CONDUCT.md | ||
| ├── GOVERNANCE.md | ||
| ├── LICENSE | ||
| ├── MAINTAINERS.md | ||
| ├── README.adoc | ||
| ├── SECURITY.md | ||
| ├── flake.guix # Guix flake (Perimeter 1) | ||
| └── Justfile # Task runner (Perimeter 1) | ||
| ``` | ||
|
|
||
| --- | ||
|
|
||
| ## How to Contribute | ||
| ## How to Contribute | ||
|
|
||
| ### Reporting Bugs | ||
| ### Reporting Bugs | ||
|
Comment on lines
+54
to
+56
Contributor
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. 🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win 🔎 Supported by static analysis🏁 Script executed: sed -n '35,110p' .github/CONTRIBUTING.md | cat -vetRepository: hyperpolymath/error-lang Length of output: 2360 Remove the four-space indentation from the section bodies. The new headings are valid Markdown headings, but the paragraphs and lists below them still begin with four spaces. Markdown renders these bodies as code blocks. Remove the indentation under the affected headings, including the ranges at lines 58, 74, 85–87, and 95. 🤖 Prompt for AI Agents |
||
|
|
||
| **Before reporting**: | ||
| 1. Search existing issues | ||
|
|
@@ -55,7 +70,7 @@ flake (Perimeter 1) └── Justfile \# Task runner (Perimeter 1) | |
| - Expected vs actual behaviour | ||
| - Logs, screenshots, or minimal reproduction | ||
|
|
||
| ### Suggesting Features | ||
| ### Suggesting Features | ||
|
|
||
| **Before suggesting**: | ||
| 1. Check the [roadmap](ROADMAP.md) if available | ||
|
|
@@ -71,7 +86,7 @@ flake (Perimeter 1) └── Justfile \# Task runner (Perimeter 1) | |
| - Alternatives considered | ||
| - Which perimeter this affects | ||
|
|
||
| ### Your First Contribution | ||
| ### Your First Contribution | ||
|
|
||
| Look for issues labelled: | ||
|
|
||
|
|
@@ -82,22 +97,25 @@ flake (Perimeter 1) └── Justfile \# Task runner (Perimeter 1) | |
|
|
||
| --- | ||
|
|
||
| ## Development Workflow | ||
| ## Development Workflow | ||
|
|
||
| ### Branch Naming | ||
| ### Branch Naming | ||
|
|
||
| docs/short-description \# Documentation (P3) test/what-added \# Test | ||
| additions (P3) feat/short-description \# New features (P2) | ||
| fix/issue-number-description \# Bug fixes (P2) refactor/what-changed \# | ||
| Code improvements (P2) security/what-fixed \# Security fixes (P1-2) | ||
| docs/short-description # Documentation (P3) test/what-added # Test | ||
| additions (P3) feat/short-description # New features (P2) | ||
| fix/issue-number-description # Bug fixes (P2) refactor/what-changed # | ||
| Code improvements (P2) security/what-fixed # Security fixes (P1-2) | ||
|
|
||
|
|
||
| ### Commit Messages | ||
| ### Commit Messages | ||
|
|
||
| We follow [Conventional Commits](https://www.conventionalcommits.org/): | ||
|
|
||
| (): | ||
| type(scope): description | ||
|
|
||
| Body: what changed and why. | ||
|
|
||
| Footer: issue reference, e.g. Closes #123 | ||
|
Comment on lines
+114
to
+118
Contributor
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: sed -n '90,110p' .github/CONTRIBUTING.mdRepository: hyperpolymath/error-lang Length of output: 593 Place optional commit-message parts in the correct order. The guide lists 🤖 Prompt for AI Agents |
||
| \[optional body\] | ||
|
|
||
| \[optional footer\] | ||
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.
🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win
🔎 Supported by static analysis
🏁 Script executed:
Repository: hyperpolymath/error-lang
Length of output: 611
Fence the complete setup sequence as shell code.
The
#lines currently render as Markdown headings, and the commands render as ordinary paragraph text. Enclose the full sequence in a fencedshblock so it renders as one copyable shell block.🤖 Prompt for AI Agents