Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Show manual test instructions in verbose mode #3431

Merged
merged 3 commits into from
Jan 16, 2025
Merged

Show manual test instructions in verbose mode #3431

merged 3 commits into from
Jan 16, 2025

Conversation

psss
Copy link
Collaborator

@psss psss commented Dec 19, 2024 •
edited
Loading

Include instructions for the manual test execution in verbose mode of the tmt test show command.

Pull Request Checklist

  • implement the feature
  • extend the test coverage
  • include a release note

@psss psss added this to the 1.41 milestone Dec 19, 2024
@happz happz modified the milestones: 1.41, 1.42 Jan 13, 2025
@psss psss requested a review from lukaszachy January 14, 2025 16:16
@psss psss added priority | should medium priority, should be included in the next release code | base Base classes implementation in base.py labels Jan 14, 2025
@happz happz added the status | ready for merge The only missing piece is to do the rebase the current 'main' and let the CI finish. label Jan 15, 2025
@happz
Copy link
Collaborator

happz commented Jan 15, 2025

@psss do you plan to address unchecked boxes of the checklist?

@psss
Copy link
Collaborator Author

psss commented Jan 16, 2025

@psss do you plan to address unchecked boxes of the checklist?

I'll add a simple test coverage and a release note.

psss added 3 commits January 16, 2025 09:57
@psss
Show manual test instructions in verbose mode
50b7fe6 
Include instructions for the manual test execution in verbose mode
of the `tmt test show` command.
@psss
Add a simple test coverage
ee604ac
@psss
Mention the change in the release notes
0ea5af4
@psss psss force-pushed the manual-test-show branch from cbd9384 to 0ea5af4 Compare January 16, 2025 09:00
@psss psss added the ci | full test Pull request is ready for the full test execution label Jan 16, 2025
@psss
Copy link
Collaborator Author

psss commented Jan 16, 2025

I'll add a simple test coverage and a release note.

Done.

@psss
Copy link
Collaborator Author

psss commented Jan 16, 2025

Rawhide failure is irrelevant.

@psss psss merged commit 1a42f5a into main Jan 16, 2025
18 of 20 checks passed
@psss psss deleted the manual-test-show branch January 16, 2025 10:22
@psss psss self-assigned this Jan 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@therazix therazix therazix approved these changes

@lukaszachy lukaszachy lukaszachy approved these changes

@happz happz Awaiting requested review from happz happz is a code owner

@thrix thrix Awaiting requested review from thrix thrix is a code owner

@janhavlin janhavlin Awaiting requested review from janhavlin janhavlin is a code owner

@martinhoyer martinhoyer Awaiting requested review from martinhoyer martinhoyer is a code owner

Assignees

@psss psss

Labels
ci | full test Pull request is ready for the full test execution code | base Base classes implementation in base.py priority | should medium priority, should be included in the next release status | ready for merge The only missing piece is to do the rebase the current 'main' and let the CI finish.
Projects
None yet
Milestone
1.42
Development

Successfully merging this pull request may close these issues.
4 participants
@psss @happz @lukaszachy @therazix