forked from data-engineering-collective/minimalkv
-
Notifications
You must be signed in to change notification settings - Fork 0
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
Auto-update pre-commit hooks #1
Open
github-actions
wants to merge
55
commits into
refactor
Choose a base branch
from
pre-commit-autoupdate
base: refactor
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains 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
56dfcb1
to
8686219
Compare
69dbd8f
to
dda10f6
Compare
d600c70
to
ce53708
Compare
…C/refactor Refactor
…ng-collective/pre-commit-autoupdate Auto-update pre-commit hooks
The previous return type of `Iterable` is incorrect as can be seen in this example: https://docs.python.org/3/library/typing.html#nominal-vs-structural-subtyping Also this documentation mentions that `__iter__` returns the iterator object itself: https://docs.python.org/3/library/stdtypes.html#iterator.__iter__ The incorrect `Iterable` type causes numerous of error when analyzed by a static type checker such as `mypy`. The switch to `Iterator` solves these issues.
…ing-iterator-typing Changing return type of `__iter__` to `Iterator`.
According to the pypi list of standardized classifiers, the classifier `License :: OSI Approved :: BSD-3 Clause` is invalid. This is also made clear by looking at the two latest publish package steps, which both failed with the following error: ``` HTTPError: 400 Bad Request from https://upload.pypi.org/legacy/ Invalid value for classifiers. Error: Classifier 'License :: OSI Approved :: BSD-3 Clause' is not a valid classifier. ``` Removing this classifier should make the package valid for publishing again as all remaining classifiers are valid. See: https://pypi.org/classifiers/
…ving-invalid-classifier Removing invalid `BSD-3 Clause` license classifier.
…ing-collective/mypy-ci Add mypy to CI
Since virtually every implementation expects the underlying data type of the file-like object to be bytes, it is more appropriate to use `BinaryIO` in favor of the broader `IO` type.
…ging-io-typing Changing generic `IO` type to `BinaryIO`.
…ecate Undeprecate url2dict and get_store_from_url
Bumps the dependencies group with 2 updates: [google-github-actions/auth](https://github.com/google-github-actions/auth) and [pypa/gh-action-pypi-publish](https://github.com/pypa/gh-action-pypi-publish). Updates `google-github-actions/auth` from 1 to 2 - [Release notes](https://github.com/google-github-actions/auth/releases) - [Changelog](https://github.com/google-github-actions/auth/blob/main/CHANGELOG.md) - [Commits](google-github-actions/auth@v1...v2) Updates `pypa/gh-action-pypi-publish` from 1.8.10 to 1.8.11 - [Release notes](https://github.com/pypa/gh-action-pypi-publish/releases) - [Commits](pypa/gh-action-pypi-publish@v1.8.10...v1.8.11) --- updated-dependencies: - dependency-name: google-github-actions/auth dependency-type: direct:production update-type: version-update:semver-major dependency-group: dependencies - dependency-name: pypa/gh-action-pypi-publish dependency-type: direct:production update-type: version-update:semver-patch dependency-group: dependencies ... Signed-off-by: dependabot[bot] <[email protected]>
dda10f6
to
796c511
Compare
…ing-collective/dependabot/github_actions/dependencies-626bd4a686
…ing-collective/pre-commit-autoupdate
Bumps the dependencies group with 2 updates: [mamba-org/setup-micromamba](https://github.com/mamba-org/setup-micromamba) and [actions/setup-python](https://github.com/actions/setup-python). Updates `mamba-org/setup-micromamba` from 1.6.0 to 1.7.0 - [Release notes](https://github.com/mamba-org/setup-micromamba/releases) - [Commits](mamba-org/setup-micromamba@db1df3b...8752438) Updates `actions/setup-python` from 4 to 5 - [Release notes](https://github.com/actions/setup-python/releases) - [Commits](actions/setup-python@v4...v5) --- updated-dependencies: - dependency-name: mamba-org/setup-micromamba dependency-type: direct:production update-type: version-update:semver-minor dependency-group: dependencies - dependency-name: actions/setup-python dependency-type: direct:production update-type: version-update:semver-major dependency-group: dependencies ... Signed-off-by: dependabot[bot] <[email protected]>
…ing-collective/dependabot/github_actions/dependencies-529127a83b
Bumps the dependencies group with 2 updates: [mamba-org/setup-micromamba](https://github.com/mamba-org/setup-micromamba) and [github/codeql-action](https://github.com/github/codeql-action). Updates `mamba-org/setup-micromamba` from 1.7.0 to 1.7.3 - [Release notes](https://github.com/mamba-org/setup-micromamba/releases) - [Commits](mamba-org/setup-micromamba@8752438...e820223) Updates `github/codeql-action` from 2 to 3 - [Release notes](https://github.com/github/codeql-action/releases) - [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md) - [Commits](github/codeql-action@v2...v3) --- updated-dependencies: - dependency-name: mamba-org/setup-micromamba dependency-type: direct:production update-type: version-update:semver-patch dependency-group: dependencies - dependency-name: github/codeql-action dependency-type: direct:production update-type: version-update:semver-major dependency-group: dependencies ... Signed-off-by: dependabot[bot] <[email protected]>
…ing-collective/dependabot/github_actions/dependencies-c85022f546 Bump the dependencies group with 2 updates
796c511
to
41a3b8e
Compare
…QC/add-session-support-s3 Allow `session_token` to be set for S3 buckets. Truly isolates stores by explicitly setting credentials for `S3FS`.
…ing-collective/dependabot/github_actions/dependencies-b275077d48 Bump the dependencies group with 1 update
Bumps the dependencies group with 2 updates: [pypa/gh-action-pypi-publish](https://github.com/pypa/gh-action-pypi-publish) and [peter-evans/create-pull-request](https://github.com/peter-evans/create-pull-request). Updates `pypa/gh-action-pypi-publish` from 1.8.11 to 1.8.12 - [Release notes](https://github.com/pypa/gh-action-pypi-publish/releases) - [Commits](pypa/gh-action-pypi-publish@v1.8.11...v1.8.12) Updates `peter-evans/create-pull-request` from 6.0.0 to 6.0.1 - [Release notes](https://github.com/peter-evans/create-pull-request/releases) - [Commits](peter-evans/create-pull-request@b1ddad2...a4f52f8) --- updated-dependencies: - dependency-name: pypa/gh-action-pypi-publish dependency-type: direct:production update-type: version-update:semver-patch dependency-group: dependencies - dependency-name: peter-evans/create-pull-request dependency-type: direct:production update-type: version-update:semver-patch dependency-group: dependencies ... Signed-off-by: dependabot[bot] <[email protected]>
…ing-collective/dependabot/github_actions/dependencies-bb80e6630a
41a3b8e
to
a09b210
Compare
…ing-collective/pre-commit-autoupdate Auto-update pre-commit hooks
Extend the minio docker setup with a custom Dockerfile that uses mc to configure 2 users with access to a different bucket each. Fix handling of url param "verify" that can be passed to [h]s3 stores.
* Add S3FSStore aws integration tests. * Allow skipping of aws integration tests when in fork or in local setup. * Improve CI by separating test and build step
* Add reason attribute to pytest skips * Refactor bucket contextmanager to avoid skipping in unnecessary cases * Fix surfacing errors if tests are not falsely skipped anymore
* Fix startup script file mode * Extend changelog
…QC/add-s3fs-integration-test Add AWS S3 integration test, add minio tests
Bumps the dependencies group with 1 update: [pypa/gh-action-pypi-publish](https://github.com/pypa/gh-action-pypi-publish). Updates `pypa/gh-action-pypi-publish` from 1.8.12 to 1.8.14 - [Release notes](https://github.com/pypa/gh-action-pypi-publish/releases) - [Commits](pypa/gh-action-pypi-publish@v1.8.12...v1.8.14) --- updated-dependencies: - dependency-name: pypa/gh-action-pypi-publish dependency-type: direct:production update-type: version-update:semver-patch dependency-group: dependencies ... Signed-off-by: dependabot[bot] <[email protected]>
…ing-collective/dependabot/github_actions/dependencies-7aaf6e754f
a09b210
to
88646e0
Compare
88646e0
to
c837c20
Compare
c837c20
to
1b32c36
Compare
1b32c36
to
787c236
Compare
787c236
to
5f6b4d1
Compare
5f6b4d1
to
4747ed6
Compare
4747ed6
to
09745d6
Compare
09745d6
to
346db25
Compare
346db25
to
b2ebf1b
Compare
b2ebf1b
to
c917f60
Compare
c917f60
to
3d82bbc
Compare
3d82bbc
to
777a906
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
New versions of the used pre-commit hooks were detected.
This PR updates them to the latest and already ran
pre-commit run -a
for you to fix any changes in formatting.