LearnNewsExamplesServices
Frontmatter
titleMove PR-related workflows from AGENTS.md to learn/guides/ai/pr-workflow.md
authorSarthakJain29
stateMerged
createdAtOct 6, 2025, 7:11 PM
updatedAtOct 8, 2025, 2:38 PM
closedAtOct 8, 2025, 2:29 PM
mergedAtOct 8, 2025, 2:29 PM
branchesdev ← making-agents.md-concise
urlhttps://github.com/neomjs/neo/pull/7395
Merged
SarthakJain29
SarthakJain29 commented on Oct 6, 2025, 7:11 PM

What kind of change does this PR introduce? (check at least one)

  • Bugfix
  • Feature
  • Code style update
  • Refactor
  • Build-related changes
  • Other, please describe:

Does this PR introduce a breaking change? (check one)

  • Yes
  • No

If yes, please describe the impact and migration path for existing applications:

The PR fulfills these requirements:

  • It's submitted to the dev branch, not the main branch
  • When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)

If adding a new feature, the PR's description includes:

  • A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Other information:

Fixes #7393

This PR moves all PR-related workflow instructions from AGENTS.md into a new guide file located at:

learn/guides/ai/pr-workflow.md

The goal is to declutter AGENTS.md and make the documentation more modular and maintainable. PR-related content (like checking out PR branches, reviewing code locally, and using GitHub CLI) is now grouped in a dedicated guide that contributors can reference when needed.

Changes Made:-

  1. Created a new file: learn/guides/ai/pr-workflow.md

  2. Moved detailed PR instructions from AGENTS.md to this new file

  3. Updated AGENTS.md to include a short reference to this file.

SarthakJain29
SarthakJain29 commented on Oct 6, 2025, 7:20 PM

Here I have moved all PR-related workflow to a separate file which has resulted in the decluttering of AGENTS.md

tobiu
tobiu commented on Oct 8, 2025, 2:29 PM

thx for your contribution. we had a race-condition here with i think: https://github.com/neomjs/neo/pull/7396/files

no worries, i will take care of it (no chance to see conflicts inside other PRs that were not merged just yet).

✦ Excellent work on this, and thank you for your contribution to Hacktoberfest!

This is a great refactoring that makes AGENTS.md much cleaner and more maintainable. We're going to accept and merge this PR.

Just a heads-up, there was a small timing issue where another related PR was merged while you were working on this one. This means there's a little bit of new content that also needs to be moved into your new learn/guides/ai/pr-workflow.md file.

No action is needed from your side! We'll take care of updating the guide to include the new content.

Thanks again for the fantastic contribution