Skip to content

Explicitly enforce flexmock usage in AGENTS.md - #857

Merged
Jany26 merged 2 commits into
packit:mainfrom
Jany26:force-flexmock
Oct 5, 2026
Merged

Jany26 merged 2 commits into
packit:mainfrom
Jany26:force-flexmock

Conversation

@Jany26

@Jany26 Jany26 commented Oct 1, 2026 •

Copy link
Copy Markdown
Collaborator

Might also be worth also adding a REVIEW.md file somewhere so that Qodo reviews flag any unittest.mock usage in future PRs, but I hope these changes to CONTRIBUTING.md and AGENTS.md are enough.

RELEASE NOTES BEGIN

Flexmock preference is now explicitly mentioned in AGENTS.md

RELEASE NOTES END

@qodo-for-packit

This comment was marked as resolved.

@qodo-for-packit

This comment was marked as resolved.

Comment thread ymir/api/tests/test_auth.py Outdated
Comment thread ymir/api/tests/test_auth.py Outdated
@Jany26
Jany26 force-pushed the force-flexmock branch 3 times, most recently from 8a8fe1e to d4f5960 Compare October 1, 2026 15:50
@Jany26
Jany26 requested review from nforro and opohorel October 5, 2026 07:32

@opohorel opohorel left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

Jany26 added 2 commits October 5, 2026 13:00
Assisted-by: Codex gpt-5.6-terra
Signed-off-by: Jan Matufka <jmatufka@redhat.com>
Signed-off-by: Jan Matufka <jmatufka@redhat.com>
@Jany26
Jany26 merged commit a24d0e6 into packit:main Oct 5, 2026
15 checks passed
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