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

Add flake8 and tmt to Github Actions #139

Merged
merged 3 commits into from
Apr 8, 2024
Merged

Add flake8 and tmt to Github Actions #139

merged 3 commits into from
Apr 8, 2024

Conversation

comps
Copy link
Contributor

@comps comps commented Apr 5, 2024

The flake8 -v / --verbose is there to show how many files were checked, as a debugging help when flake8 doesn't find anything despite obvious issues (ie. when checkout fails and it's running in an empty directory).

However, the pip version of flake8 doesn't seem to have what my Fedora version has:

flake8.checker            MainProcess     52 INFO     Checking 37 files

either way, non-verbose / verbose probably doesn't matter.

Fixes #5

Signed-off-by: Jiri Jaburek <[email protected]>
@comps comps force-pushed the add_flake8 branch 2 times, most recently from f0f8e55 to 66880d3 Compare April 5, 2024 19:07
Signed-off-by: Jiri Jaburek <[email protected]>
@comps
Copy link
Contributor Author

comps commented Apr 5, 2024

Signed-off-by: Jiri Jaburek <[email protected]>
@comps comps changed the title Add flake8 to Github Actions Add flake8 and tmt to Github Actions Apr 5, 2024
@matusmarhefka matusmarhefka merged commit 354816b into main Apr 8, 2024
3 checks passed
@matusmarhefka matusmarhefka deleted the add_flake8 branch April 8, 2024 09:54
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.

add basic CI for this repo (for PRs)
2 participants