Skip to content

5 CI updates for the devel branch: pytest, mypy, pylint, spellcheck #8

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
Open
wants to merge 12 commits into
base: devel
Choose a base branch
from
Open

5 CI updates for the devel branch: pytest, mypy, pylint, spellcheck #8

wants to merge 12 commits into from

Conversation

bernhardkaindl
Copy link
Contributor

@bernhardkaindl bernhardkaindl commented Jan 7, 2024 •
edited
Loading

Updates:

  • Rebased as topic-specific commits based on the devel branch
  • added a simple unittest for xft-runner:
    • tests/python/conftest.py: Add a pytest fixture for unit-tests
    • tests/python/test_xtf_runner.py: Add an initial unit test for xtf-runner using the pytest fixture.
  • See below for more

Base branch: andyhpp/xtf/devel:

  • runner: Update minimum python version to 2.7 [Andrew Cooper]
  • runner: Remove the use of simplejson [Andrew Cooper]
  • CI: Add basic pre-commit integration and hook into Github Actions [Andrew Cooper]
  • CI: Certain pre-commit hooks along with fixes [Andrew Cooper] (origin/devel)

Added commits: (git log --pretty=format:'- %s' -n8 --reverse):

  • .github/workflows/codeql.yml: Run in main repo only
  • CI/pre-commit: improve the message of the git-diff job
  • CI: Add initial pytest fixture and unit test for xtf-runner
  • CI: Add running python2 -m pytest in GitHub CI
  • CI: Add mypy to pre-commit and configure mypy
  • CI: Add pylint to pre-commit and configure it
  • CI: Enable spellchecking in pylint and configure it
  • xtf-runner: Refactor get_all_test_info()

andyhhp and others added 12 commits January 6, 2024 23:54
@andyhhp
runner: Update minimum python version to 2.7
6cc2a5c 
Signed-off-by: Andrew Cooper <[email protected]>
@andyhhp
runner: Remove the use of simplejson
7370097 
This became part of the standard library in Python 2.6

Signed-off-by: Andrew Cooper <[email protected]>
@bernhardkaindl @andyhhp
CI: Add basic pre-commit integration and hook into Github Actions
b0c8c5e 
Signed-off-by: Bernhard Kaindl <[email protected]>
Signed-off-by: Andrew Cooper <[email protected]>
@bernhardkaindl @andyhhp
CI: Certain pre-commit hooks along with fixes
f3a2ac8 
Fix executable permissions, and remove trailing whitespace.

Signed-off-by: Bernhard Kaindl <[email protected]>
Signed-off-by: Andrew Cooper <[email protected]>
@bernhardkaindl
.github/workflows/codeql.yml: Run in main repo only
b66cd5e 
Signed-off-by: Bernhard Kaindl <[email protected]>
@bernhardkaindl
CI/pre-commit: improve the message of the git-diff job
e3bf734 
Signed-off-by: Bernhard Kaindl <[email protected]>
@bernhardkaindl
CI: Add initial pytest fixture and unit test for xtf-runner
ee66006 
Signed-off-by: Bernhard Kaindl <[email protected]>
@bernhardkaindl
CI: Add running python2 -m pytest in GitHub CI
db73bdd 
Signed-off-by: Bernhard Kaindl <[email protected]>
@bernhardkaindl
CI: Add mypy to pre-commit and configure mypy
64f03c3 
Signed-off-by: Bernhard Kaindl <[email protected]>
@bernhardkaindl
CI: Add pylint to pre-commit and configure it
cf6c25a 
Signed-off-by: Bernhard Kaindl <[email protected]>
@bernhardkaindl
CI: Enable spellchecking in pylint and configure it
0a9bd79 
Signed-off-by: Bernhard Kaindl <[email protected]>
@bernhardkaindl
xtf-runner: Refactor get_all_test_info()
bab5526 
Signed-off-by: Bernhard Kaindl <[email protected]>
@bernhardkaindl bernhardkaindl force-pushed the devel-with-pytest-mypy-pylint branch from 6fc785a to bab5526 Compare January 7, 2024 21:20
@bernhardkaindl bernhardkaindl mentioned this pull request Jan 7, 2024
Closed
@andyhhp andyhhp force-pushed the devel branch 13 times, most recently from 056d57a to d78ad56 Compare January 13, 2024 13:54
@andyhhp andyhhp force-pushed the devel branch 3 times, most recently from 77ff43e to 5c37e0b Compare February 28, 2024 17:43
@andyhhp andyhhp force-pushed the devel branch 2 times, most recently from 89a8757 to ba7e308 Compare August 30, 2024 18:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@bernhardkaindl @andyhhp