| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
Co-authored-by: dennisdoomen <572734+dennisdoomen@users.noreply.github.com>
| } | ||
| ``` | ||
|
|
||
| - Test method names use `Succeeds_for_*` and `Fails_for_*` – avoid "Should", "When", and "Asserting" |
There was a problem hiding this comment.
No, please use fact-based names without using words like "should", "when" and "asserting". Use InlineAssertionsSpecs as reference.
Also update the other instructions files
Sorry, something went wrong.
There was a problem hiding this comment.
Updated all three instruction files in 84856fc. The test example now shows The_foo_must_satisfy_some_condition (fact statement) instead of Fails_for_foo_with_unexpected_value, and the naming guidance now reads: "Use fact-based test method names (e.g. Succeeds_for_*, The_X_must_be_Y, An_X_is_required) – avoid 'Should', 'When', and 'Asserting'", modelled on InlineAssertionsSpecs.
Sorry, something went wrong.
Co-authored-by: dennisdoomen <572734+dennisdoomen@users.noreply.github.com>
There was a problem hiding this comment.
Great!
I was just beginning to think we needed such files to avoid having to repeat to AI agents that we they need to update the api snapshot and use the new naming conventions for test methods.
Sorry, something went wrong.
@copilot please address all comments |
Sorry, something went wrong.
….Specs Co-authored-by: dennisdoomen <572734+dennisdoomen@users.noreply.github.com>
|
@copilot Can't we just use a single Agents.md file instead? |
Sorry, something went wrong.
Each tool reads from its own specific hardcoded path — Copilot reads .github/copilot-instructions.md, Claude reads CLAUDE.md, and Junie reads .junie/guidelines.md. None of them look for a generic Agents.md, so a single file wouldn't be picked up automatically by any of them. If you'd like to reduce duplication, one option is to keep a canonical Agents.md with the shared content and have the three tool-specific files contain just a brief pointer to it. But all three files would still need to exist at their expected paths. Would you like me to go that route, or keep the three self-contained files as they are? |
Sorry, something went wrong.
|
@copilot Merge the three files into a single agents.md in the root of the repo |
Sorry, something went wrong.
Co-authored-by: dennisdoomen <572734+dennisdoomen@users.noreply.github.com> Agent-Logs-Url: https://github.com/fluentassertions/fluentassertions/sessions/85b48969-a088-4741-bf0e-506c96a12f4c
Done in 383a078. All the content is now in a single agents.md at the repo root. Each tool-specific file is a one-liner pointing to it:
|
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.