Skip to content

chore: update README to remove nightly builds section#640

Merged
wizzomafizzo merged 2 commits intomainfrom
wizzomafizzo-patch-1
Apr 10, 2026
Merged

chore: update README to remove nightly builds section#640
wizzomafizzo merged 2 commits intomainfrom
wizzomafizzo-patch-1

Conversation

@wizzomafizzo
Copy link
Copy Markdown
Member

@wizzomafizzo wizzomafizzo commented Apr 10, 2026

Removed information about automated nightly builds.

Summary by CodeRabbit

  • Documentation
    • Removed outdated mention of automated nightly builds and pre-release testing from the README.
  • Chores
    • CI workflow simplified so verification and test jobs run per their standard schedules and matrices rather than being gated by a separate path-filter step.

Removed information about automated nightly builds.
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 10, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: dd2a5b6b-f7c6-4a19-b2ca-915f244ae240

📥 Commits

Reviewing files that changed from the base of the PR and between c2d5a90 and c61edd0.

📒 Files selected for processing (1)
  • .github/workflows/lint-and-test.yml
💤 Files with no reviewable changes (1)
  • .github/workflows/lint-and-test.yml

📝 Walkthrough

Walkthrough

Removed README lines advertising automated nightly builds and a Releases link; removed the changes job from the CI workflow so lint/test jobs no longer gate on changes.outputs.code and now run unconditionally per their existing conditions.

Changes

Cohort / File(s) Summary
Documentation Update
README.md
Deleted two lines advertising nightly builds and the GitHub Releases pre-release link.
CI Workflow
.github/workflows/lint-and-test.yml
Removed the changes job (path-filter step) and deleted needs.changes.outputs.code == 'true' gating so lint, quick-test, and full-test run without that conditional dependency.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

I hopped through lines both bright and slight, 🐰
Took out the nights that ran by night,
CI gates opened, tests now play—
A spotless README greets the day. ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Title check ⚠️ Warning The title mentions removing the nightly builds section from README, but the actual changeset includes significant workflow changes to lint-and-test.yml that remove conditional job execution logic. Revise the title to reflect both major changes, such as 'chore: remove nightly builds section and simplify CI job conditions' or separate into two focused pull requests.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch wizzomafizzo-patch-1

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

@wizzomafizzo wizzomafizzo changed the title Update README to remove nightly builds section chore: update README to remove nightly builds section Apr 10, 2026
Always run all checks on every PR. The Detect Changes job
caused branch protection rule violations on doc-only PRs
because skipped matrix jobs never report a status.
@sentry
Copy link
Copy Markdown

sentry bot commented Apr 10, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@wizzomafizzo wizzomafizzo merged commit 2643e27 into main Apr 10, 2026
14 checks passed
@wizzomafizzo wizzomafizzo deleted the wizzomafizzo-patch-1 branch April 10, 2026 11:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant