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.

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

Patch: AJ's pre-commit updates #49846

Merged
merged 10 commits into from
Dec 18, 2024
+46 −57
Merged

Patch: AJ's pre-commit updates #49846

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
0439ce8
format pre-commit yaml
aaronsteers Dec 17, 2024
a11e9cc
exclude airbyte_ci non-formatted java from spotless formatter
aaronsteers Dec 17, 2024
b163c3f
Update .pre-commit-config.yaml
aaronsteers Dec 17, 2024
ac11980
ci: use pre-commit in format-check
aaronsteers Dec 17, 2024
7c6ee80
ci: use newer ubuntu
aaronsteers Dec 17, 2024
4764973
ci: explicit java and python setup
aaronsteers Dec 17, 2024
168aad8
update ignore
aaronsteers Dec 17, 2024
3bce6b3
ci: update format-fix
aaronsteers Dec 17, 2024
4ff1e85
dummy change to break formatting
aaronsteers Dec 17, 2024
aad2e23
chore: auto-fix lint and format issues
octavia-squidington-iii Dec 17, 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
11 changes: 6 additions & 5 deletions .pre-commit-config.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,18 +4,19 @@ exclude: |
^.*/__init__\.py$|
^.*?/\.venv/.*$|
^.*?/node_modules/.*$|
^.*?/charts/.*$|
^airbyte-integrations/bases/base-normalization/.*$|
^.*?/normalization_test_output/.*$|
^.*?/pnpm-lock\.yaml$|
^.*?/source-amplitude/unit_tests/api_data/zipped\.json$|
Copy link
Contributor

Choose a reason for hiding this comment

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

oooof
thank you

# Generated/test files
^airbyte-ci/connectors/metadata_service/lib/metadata_service/models/generated/.*$|
^.*?/airbyte-ci/connectors/metadata_service/lib/tests/fixtures/.*/invalid/.*$|
^airbyte-ci/connectors/pipelines/tests/test_format/non_formatted_code/.*$|
^.*?/airbyte-ci/connectors/pipelines/tests/test_format/non_formatted_code/.*$|
^.*?/airbyte-integrations/connectors/destination-.*/expected-spec\.json$
)
Expand All @@ -25,15 +26,15 @@ repos:
hooks:
- id: black
language_version: python3.10
verbose: true
args: ["--verbose"]
args: [--config=pyproject.toml]
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

The --config-pyproject.toml arg might not be needed, but consistency seemed worthwhile.

Also - I'm okay bringing back verbose if we need it, but was very spammy on my machine when --all-files was passed as an arg.

(I also tried sending an explicit config path to isort - but this had an adverse affect so I reverted it.)


- repo: https://github.com/pycqa/isort
rev: 5.12.0
hooks:
- id: isort
files: \.py$
require_serial: true
# args: [--settings-file=pyproject.toml]
aaronsteers marked this conversation as resolved.
Show resolved Hide resolved

- repo: https://github.com/pre-commit/mirrors-prettier
rev: v3.0.3
Expand Down
3 changes: 3 additions & 0 deletions spotless-maven-pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,9 @@
<includes>
<include>**/*.java</include>
</includes>
<excludes>
<exclude>**/non_formatted_code/*</exclude>
</excludes>
Comment on lines +29 to +31
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This was necessary because this file would otherwise be formatted by spotless when running pre-commit with the --all-files flag.

Since spotless doesn't receive individual files from pre-commit, the pre-commit exclude list doesn't apply.

<importOrder />
<eclipse>
<version>4.21</version>
Expand Down
Loading