Skip to content

Add instructions for running focused checks on integration packages - #967

Merged
dahlia merged 1 commit into
fedify-dev:mainfrom
Bananamilk452:issue-890-document-focused-checks-for-integration-packages
Jul 24, 2026
Merged

Add instructions for running focused checks on integration packages#967
dahlia merged 1 commit into
fedify-dev:mainfrom
Bananamilk452:issue-890-document-focused-checks-for-integration-packages

Conversation

@Bananamilk452

@Bananamilk452 Bananamilk452 commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Close #890

This pull request updates the docs/manual/integration.md file to provide guidance on running focused checks when making changes to integration packages. The documentation now includes an example command for running checks on a specific package.

Documentation improvements:

  • Added instructions on how to run focused checks for integration packages, including an example command for checking @fedify/express after making changes.

* OpenCode:gpt-5.6-terra helped with docs writing.

@gemini-code-assist

Copy link
Copy Markdown
Contributor

Caution

The consumer version of Gemini Code Assist on GitHub has been sunset. All code review activity has officially ceased.

@netlify

netlify Bot commented Jul 24, 2026

Copy link
Copy Markdown

Deploy Preview for fedify-json-schema canceled.

Name Link
🔨 Latest commit e3917fd
🔍 Latest deploy log https://app.netlify.com/projects/fedify-json-schema/deploys/6a632656faa87100083bab81

@coderabbitai

coderabbitai Bot commented Jul 24, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

The integration manual now instructs contributors to run a focused package check when modifying an integration package, with a mise run check-each example.

Changes

Integration check guidance

Layer / File(s) Summary
Focused check guidance
docs/manual/integration.md
Adds instructions and an example command for running the focused check corresponding to a changed integration package.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related issues

  • fedify-dev/fedify#890 — Documents package-level focused checks in docs/manual/integration.md.

Suggested reviewers: 2chanhaeng

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title clearly summarizes the main change: adding focused check instructions for integration packages.
Description check ✅ Passed The description matches the documented update and mentions the focused check guidance and example command.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@docs/manual/integration.md`:
- Line 16: Update the package specifier in the documentation sentence near “For
example, after changing” to use inline code formatting instead of emphasis,
preserving the surrounding wording.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: a097d091-9f6a-4372-9897-a17e5fbc4731

📥 Commits

Reviewing files that changed from the base of the PR and between c2352c0 and cb5fbf7.

📒 Files selected for processing (1)
  • docs/manual/integration.md

Comment thread docs/manual/integration.md
@codecov

codecov Bot commented Jul 24, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.
see 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@Bananamilk452
Bananamilk452 marked this pull request as draft July 24, 2026 08:35
@Bananamilk452
Bananamilk452 force-pushed the issue-890-document-focused-checks-for-integration-packages branch from cb5fbf7 to e3917fd Compare July 24, 2026 08:46
@Bananamilk452
Bananamilk452 marked this pull request as ready for review July 24, 2026 08:47
@gemini-code-assist

Copy link
Copy Markdown
Contributor

Caution

The consumer version of Gemini Code Assist on GitHub has been sunset. All code review activity has officially ceased.

@dahlia dahlia self-assigned this Jul 24, 2026
@dahlia dahlia added the component/integration Web framework integration label Jul 24, 2026
@dahlia dahlia added this to the Fedify 2.4 milestone Jul 24, 2026

@dahlia dahlia left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@dahlia
dahlia merged commit b4ee305 into fedify-dev:main Jul 24, 2026
25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component/integration Web framework integration

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Document focused checks for integration packages

2 participants