Skip to content

Add missing main.fmf for copr-repository artifact provider test#4720

Merged
psss merged 1 commit intomainfrom
avinay-copr-repository-missing-main-fmf
Mar 20, 2026
Merged

Add missing main.fmf for copr-repository artifact provider test#4720
psss merged 1 commit intomainfrom
avinay-copr-repository-missing-main-fmf

Conversation

@AthreyVinay
Copy link
Copy Markdown
Member

@AthreyVinay AthreyVinay commented Mar 19, 2026

No description provided.

gemini-code-assist[bot]
gemini-code-assist Bot reviewed Mar 19, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request adds a main.fmf metadata file for the copr-repository artifact provider test. The change is correct, but I have a suggestion to improve the clarity of the test description by removing redundancy.

Comment thread tests/prepare/artifact/providers/copr-repository/main.fmf
@LecrisUT LecrisUT added test coverage Improvements or additions to test coverage of tmt itself ci | full test Pull request is ready for the full test execution review | trivial Very easy for review, even for beginners, so don't be afraid to have a look! :-) plugin | artifact Related to the `prepare/artifact` plugin. labels Mar 19, 2026
@github-project-automation github-project-automation Bot moved this to backlog in planning Mar 19, 2026
@LecrisUT LecrisUT added the status | blocking other work An important pull request, blocking other pull requests or issues label Mar 19, 2026
@LecrisUT LecrisUT moved this from backlog to review in planning Mar 19, 2026
@LecrisUT LecrisUT added this to the 1.70 milestone Mar 19, 2026
@LecrisUT LecrisUT moved this from review to merge in planning Mar 19, 2026
@psss psss merged commit 11e3207 into main Mar 20, 2026
33 checks passed
@psss psss deleted the avinay-copr-repository-missing-main-fmf branch March 20, 2026 08:35
@github-project-automation github-project-automation Bot moved this from merge to done in planning Mar 20, 2026
@AthreyVinay AthreyVinay mentioned this pull request Mar 23, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@psss psss psss approved these changes

@vaibhavdaren vaibhavdaren vaibhavdaren approved these changes

@LecrisUT LecrisUT LecrisUT approved these changes

+1 more reviewer

@gemini-code-assist gemini-code-assist[bot] gemini-code-assist[bot] left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

ci | full test Pull request is ready for the full test execution plugin | artifact Related to the `prepare/artifact` plugin. review | trivial Very easy for review, even for beginners, so don't be afraid to have a look! :-) status | blocking other work An important pull request, blocking other pull requests or issues test coverage Improvements or additions to test coverage of tmt itself

Projects

planning
Status: done

Milestone

1.70

Development

Successfully merging this pull request may close these issues.

4 participants

@AthreyVinay @psss @vaibhavdaren @LecrisUT