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

Add a new Execute Tests section to the Guide #3086

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Add a new Execute Tests section to the Guide #3086

wants to merge 1 commit into from

Conversation

psss
Copy link
Collaborator

@psss psss commented Jul 12, 2024

Move the Run section from examples to the guide. Also move the misplaced Rerunning failed tests section here.

Pull Request Checklist

  • implement the feature
  • write the documentation
  • extend the test coverage
  • update the specification
  • adjust plugin docstring
  • modify the json schema
  • mention the version
  • include a release note

@psss psss added the documentation Improvements or additions to documentation label Jul 12, 2024
lukaszachy
lukaszachy reviewed Jul 12, 2024
View reviewed changes
docs/guide.rst
------------------------------------------------------------------

Executing failed tests again after fixing them is now possible
with `tmt run --all --again tests --failed-only`.
Copy link
Collaborator

Choose a reason for hiding this comment

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

How about adding --last in the example? Otherwise it's not possible to copy&paste. Sure, you mention that below but it will be easier if the example was a working one already.

@psss
Add a new Execute Tests section to the Guide
0fa32e4 
Move the `Run` section from examples to the guide.
Also move the misplaced `Rerunning failed tests` section here.
@psss psss force-pushed the guide-execute-tests branch from 5c59179 to 0fa32e4 Compare July 16, 2024 07:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lukaszachy lukaszachy lukaszachy left review comments

@happz happz Awaiting requested review from happz happz will be requested when the pull request is marked ready for review happz is a code owner

@thrix thrix Awaiting requested review from thrix thrix will be requested when the pull request is marked ready for review thrix is a code owner

@janhavlin janhavlin Awaiting requested review from janhavlin janhavlin will be requested when the pull request is marked ready for review janhavlin is a code owner

@martinhoyer martinhoyer Awaiting requested review from martinhoyer martinhoyer will be requested when the pull request is marked ready for review martinhoyer is a code owner

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
documentation Improvements or additions to documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@psss @lukaszachy