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.

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

Improve README section for running tests #199

Merged
merged 1 commit into from
Nov 17, 2021

Conversation

Martchus
Copy link
Contributor

  • Add more details about tox environments
  • Mention how to ensure correct formatting of files
  • Keep information related to direct pytest invocations together (and
    not mention how to generate the coverage report in the middle)
  • Use subsections for the different parts

README.md Outdated Show resolved Hide resolved
@Martchus Martchus force-pushed the readme branch 3 times, most recently from 97cdc30 to 30db5bd Compare November 17, 2021 15:02
@Martchus
Copy link
Contributor Author

@perlpunk I've changed the black command back because the tox -e black actually only checks whether the source files are correctly formatted. However, here I want to show the command for actually formatting them.

Copy link
Member

@okurz okurz left a comment

Choose a reason for hiding this comment

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

one trivial question, rest is fine, +1 in general

README.md Outdated Show resolved Hide resolved
* Add more details about `tox` environments
* Mention how to workaround installation issues in `tox` environments
* Mention how to ensure correct formatting of files
* Keep information related to direct `pytest` invocations together (and
  not mention how to generate the coverage report in the middle)
* Use subsections for the different parts
@Martchus Martchus merged commit 6962af8 into os-autoinst:master Nov 17, 2021
@Martchus Martchus deleted the readme branch November 17, 2021 16:36
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.

None yet

3 participants