Skip to content

Update agents.md: remove stale synctest reference, minor cleanups#19415

Closed
anacrolix wants to merge 2 commits intomainfrom
anacrolix/claude-rules
Closed

Update agents.md: remove stale synctest reference, minor cleanups#19415
anacrolix wants to merge 2 commits intomainfrom
anacrolix/claude-rules

Conversation

@anacrolix
Copy link
Copy Markdown
Contributor

@anacrolix anacrolix commented Feb 23, 2026

Summary

  • Remove goexperiment.synctest build tag reference (dropped from project)
  • "Full test suite with coverage""Full test suite" (coverage not implied by make test-all)
  • Reorder to put run-specific-tests before the pre-commit reminder
  • Clean up lint section heading and redundant (make lint) parenthetical

@anacrolix
Copy link
Copy Markdown
Contributor Author

I extracted the rules per claude recommendations, but then discovered that AGENTS.md is a new practice for general AI use. I'll find out what the plan is first.

Remove the synctest build tag reference (dropped from project). Reorder
to put run-specific-tests before the pre-commit reminder. Minor wording
cleanups to lint and build sections.
@anacrolix anacrolix force-pushed the anacrolix/claude-rules branch from f4b3d8a to f4cccd4 Compare February 23, 2026 01:03
@anacrolix anacrolix changed the title .claude: refactor CLAUDE.md into modular rules files Update agents.md: remove stale synctest reference, minor cleanups Feb 23, 2026
@Giulio2002
Copy link
Copy Markdown
Collaborator

[SharovBot] 🤖 Closed by automated review.

This PR appears to be an AI-generated stylistic/convention change (dead code removal, unused variable cleanup, redundant code removal) from a non-core contributor.

These types of changes are not being accepted at this time. The changes will be included in a consolidated weekly cleanup PR. If you believe this was closed in error, please reach out to a maintainer.

@Giulio2002 Giulio2002 closed this Feb 23, 2026
@Giulio2002 Giulio2002 reopened this Feb 23, 2026
@Giulio2002
Copy link
Copy Markdown
Collaborator

SHAHHAHAAHAHAHA

@anacrolix
Copy link
Copy Markdown
Contributor Author

I'll clean this up shortly

@anacrolix anacrolix self-assigned this Mar 4, 2026
@anacrolix
Copy link
Copy Markdown
Contributor Author

The remaining changes only make sense in combination with #19223.

@anacrolix anacrolix closed this Mar 6, 2026
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.

2 participants