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

DAOS-623 ci: Add config and Action for python formatting. #13281

Closed
wants to merge 33 commits into from
Closed
Show file tree
Hide file tree
Changes from 6 commits
Commits
Show all changes
33 commits
Select commit Hold shift + click to select a range
3a4c9c6
DAOS-623 ci: Add config and Action for python formatting.
ashleypittman Nov 2, 2023
a63a134
Change options.
ashleypittman Nov 2, 2023
787a57a
Change some things.
ashleypittman Nov 2, 2023
2a4a5dc
Change more files.
ashleypittman Nov 2, 2023
1036d1e
Fix pylint.
ashleypittman Nov 2, 2023
d3fa8be
Rename a file.
ashleypittman Nov 2, 2023
9996dc3
Add a new package.
ashleypittman Nov 2, 2023
610ebaf
Revert changes to utils, except using/sl
ashleypittman Nov 3, 2023
2934722
Back out conflict.
ashleypittman Nov 3, 2023
0120f32
Merge branch 'master' into amd/black
ashleypittman Nov 3, 2023
98e1fff
Fix isort errors.
ashleypittman Nov 3, 2023
2fb4469
Merge branch 'master' into amd/black
ashleypittman Nov 6, 2023
c6904f5
Add a package for pylint
ashleypittman Nov 6, 2023
74056ef
back out cahge.
ashleypittman Nov 13, 2023
a8161c7
Required-githooks: true
ashleypittman Nov 13, 2023
b6613c1
Resolve conflicts.
ashleypittman Nov 17, 2023
9b1a448
Merge branch 'master' into amd/black
ashleypittman Nov 17, 2023
1160bfb
make check it's own chek.
ashleypittman Nov 17, 2023
f8527f0
Revise code.
ashleypittman Nov 17, 2023
53c207c
Fix black check.
ashleypittman Nov 17, 2023
90d3930
Merge branch 'master' into amd/black
ashleypittman Nov 24, 2023
1f9bd95
Fix file.
ashleypittman Nov 24, 2023
da1bb8d
Add --diff option.
ashleypittman Nov 24, 2023
a41d093
Fix conversion file.
ashleypittman Nov 24, 2023
aa68870
Back out conflict.
ashleypittman Dec 8, 2023
2e56389
Merge branch 'master' into amd/black
ashleypittman Dec 8, 2023
639ffe2
Fix the two files.
ashleypittman Dec 8, 2023
6d954ee
Back out raft cahnge
ashleypittman Dec 8, 2023
0a6a0c3
Merge branch \'amd/black\' into origin/master
ashleypittman Jan 16, 2024
a3d21e3
Back out conflict.
ashleypittman Jan 30, 2024
1b4898c
Merge branch 'master' into amd/black
ashleypittman Jan 30, 2024
631de86
Patch build file.
ashleypittman Jan 30, 2024
c02152f
Update to 2024 stlye.
ashleypittman Jan 30, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .flake8
Original file line number Diff line number Diff line change
Expand Up @@ -13,4 +13,4 @@ exclude =
build,
install
max-line-length: 100
ignore=W503
ignore=W503,E203
2 changes: 1 addition & 1 deletion .flake8-scons
Original file line number Diff line number Diff line change
Expand Up @@ -13,4 +13,4 @@ exclude =
install
filename=*/SConscript, SConstruct
max-line-length: 100
ignore=F821,F841,W503
ignore=F821,F841,W503,E203
4 changes: 2 additions & 2 deletions .github/workflows/flake.yml
Original file line number Diff line number Diff line change
Expand Up @@ -28,12 +28,12 @@ jobs:
with:
# W503 and W504 are related as they conflict. W503 is the preferred style and all code
# should be using it now.
ignore: 'W503'
ignore: 'W503,E203'
exclude: 'src/control/vendor,src/client/pydaos/raw'
max-line-length: '100'
- name: flake8 Lint on SCons files.
uses: py-actions/flake8@v2
with:
ignore: 'F821,W503,F841'
ignore: 'F821,W503,F841,E203'
max-line-length: '100'
args: '--filename */SConscript, SConstruct'
4 changes: 4 additions & 0 deletions .github/workflows/linting.yml
Original file line number Diff line number Diff line change
Expand Up @@ -40,3 +40,7 @@ jobs:
ref: ${{ github.event.pull_request.head.sha }}
- name: Check DAOS logging macro use.
run: ./utils/cq/d_logging_check.py --github src
- name: Black
uses: psf/black@stable
with:
options: "--check --verbose --extend-exclude (ftest|vendor)"
Loading