-
Notifications
You must be signed in to change notification settings - Fork 31
First crack at default config for ruff
- Step 3.5 - ESCU 5.0
#254
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
Merged
Merged
Changes from all commits
Commits
Show all changes
34 commits
Select commit
Hold shift + click to select a range
2a5663d
First crack at default config
ljstella 3f7a585
Adding suggested extension config
ljstella 0a88668
Bumping target_version
ljstella 82b8fa7
Merge branch 'main' into ruff_config
ljstella 5a9af4f
Merge branch 'main' into ruff_config
ljstella 2568f71
Updated pyproject.toml
ljstella ec40b40
Merge branch 'main' into ruff_config
ljstella 0b4158b
Github CI
ljstella f99655d
Add precommit hook
ljstella c1bdfbc
Update gitignore
ljstella 7ed5e02
Updating ruff workflow
ljstella 9aa1607
Adding ruff as dev dependency
ljstella 9a493c6
Merge branch 'main' into ruff_config
ljstella 66ef7fc
Merge branch 'main' into ruff_config
ljstella 9ad0d96
Merge branch 'main' into ruff_config
ljstella 8f1845e
Merge branch 'main' into ruff_config
ljstella a5cd630
Merge branch 'main' into ruff_config
ljstella 45a6cb0
Merge branch 'main' into ruff_config
ljstella 82d4f16
Merge branch 'main' into ruff_config
ljstella 030ae92
bumped version
ljstella ae8c9c8
Merge branch 'main' into ruff_config
ljstella e41e427
Merge branch 'main' into ruff_config
ljstella f8d31fe
Merge branch 'main' into ruff_config
ljstella 5591952
Ruff version bump
ljstella ed1c8b0
Version bump
ljstella c0d440c
other precommit hooks
ljstella 356317f
Merge branch 'contentctl_5' into ruff_config
ljstella d71674f
version bump
ljstella 6aebe3f
Version bump
ljstella 6f60e75
version update
ljstella 8795e97
bump to 0.9.0
ljstella 633f0d5
version bump to 0.9.1
ljstella c4a88f5
Bump for 0.9.2
ljstella 3530006
Merge branch 'contentctl_5' into ruff_config
josehelps File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,22 @@ | ||
name: lint & format | ||
on: | ||
pull_request: | ||
types: [opened, reopened, synchronize] | ||
|
||
jobs: | ||
lint: | ||
runs-on: ubuntu-latest | ||
steps: | ||
- uses: actions/checkout@v4 | ||
- name: Install Python | ||
uses: actions/setup-python@v5 | ||
with: | ||
python-version: "3.11" | ||
- name: Install ruff | ||
run: | | ||
python -m pip install --upgrade pip | ||
pip install ruff | ||
- name: Run lint | ||
run: ruff check --output-format=github contentctl/ | ||
- name: Run Formatter | ||
run: ruff format --check contentctl/ |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -3,6 +3,7 @@ poetry.lock | |
# usual mac files | ||
.DS_Store | ||
*/.DS_Store | ||
.ruff_cache | ||
|
||
# custom | ||
dist/* | ||
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,16 @@ | ||
repos: | ||
- repo: https://github.com/pre-commit/pre-commit-hooks | ||
rev: v5.0.0 # Use the ref you want to point at | ||
hooks: | ||
- id: check-json | ||
- id: check-symlinks | ||
- id: check-yaml | ||
- id: detect-aws-credentials | ||
- id: detect-private-key | ||
- id: forbid-submodules | ||
- repo: https://github.com/astral-sh/ruff-pre-commit | ||
rev: v0.9.2 | ||
hooks: | ||
- id: ruff | ||
args: [ --fix ] | ||
- id: ruff-format |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,5 @@ | ||
{ | ||
"recommendations": [ | ||
"charliermarsh.ruff" | ||
] | ||
} |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.