-
-
Notifications
You must be signed in to change notification settings - Fork 0
fix(docs): repair the contributing guide content defects #208
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/developer-ecosystem.git> cd | ||
| developer-ecosystem | ||
| git clone https://github.com/hyperpolymath/developer-ecosystem.git | ||
| cd developer-ecosystem | ||
|
|
||
| # Using Nix (recommended for reproducibility) | ||
| # Using Guix (recommended for reproducibility) | ||
|
|
||
| nix develop | ||
| guix develop | ||
|
coderabbitai[bot] marked this conversation as resolved.
|
||
|
|
||
| # Or using toolbox/distrobox | ||
|
|
||
| toolbox create developer-ecosystem-dev toolbox enter | ||
| developer-ecosystem-dev \# Install dependencies manually | ||
| toolbox create developer-ecosystem-dev | ||
| toolbox enter developer-ecosystem-dev | ||
| # Install dependencies manually | ||
|
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 '1,55p' .github/CONTRIBUTING.md
rg -n 'toolbox|distrobox|dependencies|dependency|guix develop|just check|just test' README* .github docs Justfile justfile guix.scm manifest.scm flake.nix 2>/dev/nullRepository: hyperpolymath/developer-ecosystem Length of output: 2663 🏁 Script executed: #!/bin/bash
set -u
printf '%s\n' '--- candidate files ---'
git ls-files | rg '(^|/)(README[^/]*|Justfile|justfile|flake\.nix|guix\.scm|manifest\.scm|Cargo\.toml|mix\.exs|package\.json|pyproject\.toml|Gemfile|go\.mod|Makefile|CONTRIBUTING\.md)$|(^|/)docs/' | head -200
printf '%s\n' '--- CONTRIBUTING with line numbers ---'
sed -n '1,28p' .github/CONTRIBUTING.md
printf '%s\n' '--- setup/dependency references in tracked docs and manifests ---'
rg -n -i 'install|dependenc|toolbox|distrobox|guix develop|just (check|test)|cargo (check|test)|mix (compile|test)|npm (install|ci)|pnpm|yarn|pip install|bundle install|go test' \
README* .github docs Justfile justfile flake.nix guix.scm manifest.scm Cargo.toml mix.exs package.json pyproject.toml Gemfile go.mod Makefile 2>/dev/null || true
printf '%s\n' '--- Justfile relevant sections ---'
sed -n '120,160p' Justfile 2>/dev/null || trueRepository: hyperpolymath/developer-ecosystem Length of output: 18846 🤖 get_repo_knowledge executed:
Length of output: 47780 Document the toolbox dependency installation. After 🧰 Tools🪛 markdownlint-cli2 (0.23.2)[warning] 14-14: Headings should be surrounded by blank lines (MD022, blanks-around-headings) 🤖 Prompt for AI Agents |
||
|
|
||
| # Verify setup | ||
|
|
||
| just check \# or: cargo check / mix compile / etc. just test \# Run test | ||
| suite | ||
|
|
||
|
|
||
| ### Repository Structure | ||
|
|
||
| developer-ecosystem/ ├── 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.nix \# Nix | ||
| flake (Perimeter 1) └── Justfile \# Task runner (Perimeter 1) | ||
|
|
||
| just check # or: cargo check / mix compile / etc. | ||
| just test # Run test suite | ||
|
|
||
| ### Repository Structure | ||
|
|
||
| ```text | ||
| developer-ecosystem/ | ||
| ├── 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.nix # Nix flake (Perimeter 1) | ||
| └── Justfile # Task runner (Perimeter 1) | ||
| ``` | ||
|
|
||
| --- | ||
|
|
||
| ## How to Contribute | ||
| ## How to Contribute | ||
|
|
||
| ### Reporting Bugs | ||
| ### Reporting Bugs | ||
|
|
||
| **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 | ||
| \[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/developer-ecosystem
Length of output: 50389
🏁 Script executed:
Repository: hyperpolymath/developer-ecosystem
Length of output: 50391
🤖 get_repo_knowledge executed:
get_repo_knowledge hyperpolymath/developer-ecosystem /tmp/coderabbit-repo-knowledge/hyperpolymath-developer-ecosystem-676f233f/conventionsLength of output: 6904
🏁 Script executed:
Repository: hyperpolymath/developer-ecosystem
Length of output: 6038
🌐 Web query:
official GNU Guix manual guix develop no guix.scm current directory default discovery error💡 Result:
<search_synthesis>
The automatic discovery of guix.scm or manifest.scm files by guix shell is a convenience feature designed for interactive use [1][2]. If you are encountering an error or the file is not being loaded, it is likely due to one of the following reasons: 1. Non-interactive usage: The automatic loading of guix.scm or manifest.scm only occurs when guix shell is invoked without any other arguments (e.g., just running guix shell) [2][3]. If you provide additional arguments—such as packages to install, a command to execute (e.g., guix shell -- make), or other flags—guix shell enters a non-interactive mode and will ignore any local guix.scm or manifest.scm files [4][3]. 2. Authorization requirement: For security reasons, guix shell will only automatically load these files if the directory containing them is explicitly authorized [1][2]. If the directory is not authorized, guix shell will report an error [5]. To authorize a directory, you must add its absolute path to the file ~/.config/guix/shell-authorized-directories [4][5]. You can do this by running: echo /path/to/your/project >> ~/.config/guix/shell-authorized-directories 3. Precedence: If both manifest.scm and guix.scm exist in the same directory, manifest.scm takes precedence [6][2]. Guix will process the manifest.scm and ignore the guix.scm [6]. If you need to use a guix.scm file while also providing other arguments or running a command, you must explicitly specify the file using the --file option [6]. Note that when using --file with a guix.scm, you may also need to include the --development flag to ensure you are installing the package&
#39;s development dependencies rather than the package itself [6].</search_synthesis>
<source_evidence>
Citations:
🏁 Script executed:
Repository: hyperpolymath/developer-ecosystem
Length of output: 15407
🌐 Web query:
site:guix.gnu.org/manual guix develop command GNU Guix💡 Result:
<search_synthesis>
There is no command named guix develop in GNU Guix. The functionality for creating development environments is provided by the guix shell command [1][2]. Historically, the guix environment command was used for this purpose, but it has been deprecated in favor of guix shell, which is designed to be more convenient for creating one-off software environments [3]. guix shell allows users to spawn a shell with specific packages available without modifying their user profile [1][2].
</search_synthesis>
<source_evidence>
No source excerpts or source summaries were returned.
</source_evidence>
Citations:
Add a root Guix environment and use a supported command.
guix developis not a GNU Guix command. The repository root also has noguix.scmormanifest.scmfor Guix to load. Add a rootguix.scm, then replace the command with:The nested Guix files do not apply after
cd developer-ecosystem. No documented prerequisite supplies a parent environment.🤖 Prompt for AI Agents