Skip to content

docs: expand CONTRIBUTING.md with BST build context and AGENTS.md reference - #776

Merged
castrojo merged 1 commit into
mainfrom
fix/hive-contributing-update
Jun 10, 2026
Merged

docs: expand CONTRIBUTING.md with BST build context and AGENTS.md reference#776
castrojo merged 1 commit into
mainfrom
fix/hive-contributing-update

Conversation

@castrojo

Copy link
Copy Markdown
Contributor

Addresses hive advisory finding from projectbluefin/common#557.

CONTRIBUTING.md was 3–5 lines with no BST context. A contributor arriving from mainline Bluefin would try to edit a Containerfile. The critical "not-bluefin" context lived only in AGENTS.md and docs/skills/not-bluefin.md, not in the human-facing CONTRIBUTING.md.

Changes

  • Added prominent warning that dakota uses BuildStream 2, not Containerfiles
  • Added BST prerequisites (bst, podman, just)
  • Added PR target (testing branch)
  • Added repo structure overview
  • Added explicit pointer to AGENTS.md (where all build instructions live)

…erence

Contributors arriving from mainline Bluefin had no indication that dakota uses
BuildStream 2 (not Containerfiles/DNF) and no path to the actual build instructions.
This adds the critical context and points to AGENTS.md where everything lives.

Addresses hive advisory finding in projectbluefin/common#557

Assisted-by: Claude Sonnet 4.5 via pi
@castrojo
castrojo requested review from p5 and tulilirockz as code owners June 10, 2026 18:22
@coderabbitai

coderabbitai Bot commented Jun 10, 2026

Copy link
Copy Markdown

Warning

Review limit reached

@castrojo, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 15 minutes and 59 seconds. Learn how PR review limits work.

Your organization has run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 90eb6343-7374-4794-a863-4ebbaff3e127

📥 Commits

Reviewing files that changed from the base of the PR and between 2dc523b and 766e762.

📒 Files selected for processing (1)
  • CONTRIBUTING.md
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/hive-contributing-update

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions

Copy link
Copy Markdown
Contributor

Thanks for the PR! A maintainer will review it.

While you wait, make sure these pass locally:

just validate        # element graph check
just build default   # build the image
just boot-test       # confirm the desktop boots (exits 0 = pass)
just lint            # bootc container lint

If this PR fixes a bug, add verify steps to the linked issue so users can confirm the fix on their hardware after the next nightly ships:

```verify
ujust <something>   # what users should run to confirm the fix
```

@castrojo
castrojo merged commit 6b7ea1d into main Jun 10, 2026
21 checks passed
@castrojo
castrojo deleted the fix/hive-contributing-update branch June 10, 2026 18:22
@castrojo castrojo added 4-review A pull request is awaiting review. and removed pr/needs-review labels Jul 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4-review A pull request is awaiting review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant