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

Jump to bottom

⬆ [pre-commit.ci] pre-commit autoupdate #482

Merged
merged 3 commits into from
Sep 6, 2023
Merged

⬆ [pre-commit.ci] pre-commit autoupdate #482

merged 3 commits into from
Sep 6, 2023

Conversation

pre-commit-ci[bot]
Copy link
Contributor

@pre-commit-ci pre-commit-ci bot commented Aug 22, 2023 •
edited
Loading

@pre-commit-ci pre-commit-ci bot requested a review from a team as a code owner August 22, 2023 00:37
@pre-commit-ci pre-commit-ci bot requested a review from a team August 22, 2023 00:37
@netlify
Copy link

netlify bot commented Aug 22, 2023 •
edited
Loading

👷 Deploy Preview for amazing-pothos-a3bca0 processing.

Name Link
🔨 Latest commit d3ee131
🔍 Latest deploy log https://app.netlify.com/sites/amazing-pothos-a3bca0/deploys/64f8681907885a00085b31ff

@pre-commit-ci pre-commit-ci bot temporarily deployed to internal August 22, 2023 00:37 Inactive
@codecov
Copy link

codecov bot commented Aug 22, 2023 •
edited
Loading

Codecov Report

Patch coverage has no change and project coverage change: +0.03% 🎉

Comparison is base (033a4b3) 91.48% compared to head (8260b02) 91.51%.
Report is 1 commits behind head on main.

❗ Current head 8260b02 differs from pull request most recent head d3ee131. Consider uploading reports for the commit d3ee131 to get more accurate results

Additional details and impacted files 
@@            Coverage Diff             @@
##             main     #482      +/-   ##
==========================================
+ Coverage   91.48%   91.51%   +0.03%     
==========================================
  Files          49       50       +1     
  Lines        1914     1874      -40     
==========================================
- Hits         1751     1715      -36     
+ Misses        163      159       -4

see 8 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@tatiana tatiana force-pushed the pre-commit-ci-update-config branch from ff4afeb to 85c4cd0 Compare August 22, 2023 07:26
@tatiana tatiana temporarily deployed to internal August 22, 2023 07:26 — with GitHub Actions Inactive
@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch from 85c4cd0 to c32be89 Compare August 29, 2023 00:12
@pre-commit-ci pre-commit-ci bot temporarily deployed to internal August 29, 2023 00:12 Inactive
@pre-commit-ci
⬆ [pre-commit.ci] pre-commit autoupdate
8260b02 
updates:
- [github.com/astral-sh/ruff-pre-commit: v0.0.284 → v0.0.287](astral-sh/ruff-pre-commit@v0.0.284...v0.0.287)
- [github.com/asottile/blacken-docs: 1.15.0 → 1.16.0](adamchainz/blacken-docs@1.15.0...1.16.0)
- [github.com/pre-commit/mirrors-mypy: v1.5.0 → v1.5.1](pre-commit/mirrors-mypy@v1.5.0...v1.5.1)
@pre-commit-ci pre-commit-ci bot temporarily deployed to internal September 5, 2023 00:47 Inactive
@tatiana tatiana temporarily deployed to internal September 5, 2023 08:54 — with GitHub Actions Inactive
@tatiana tatiana temporarily deployed to internal September 5, 2023 09:30 — with GitHub Actions Inactive
tatiana added a commit that referenced this pull request Sep 5, 2023
@tatiana
Enable MyPy checks in the CI, since pre-commit reached quota limit
aeba36d 
As observed in:
#482

The following exception was raised:
build of https://github.com/pre-commit/mirrors-mypy:types-PyYAML,types-attrs,attrs,types-requests,types-python-dateutil,[email protected] for python@python3 exceeds tier max size 250MiB: 262.2MiB

So we disabled MyPy checks as part of #485
@tatiana tatiana mentioned this pull request Sep 5, 2023
Merged
tatiana added a commit that referenced this pull request Sep 6, 2023
@tatiana
Enable MyPy checks in the CI, since pre-commit reached quota limit
ba58859 
As observed in:
#482

The following exception was raised:
build of https://github.com/pre-commit/mirrors-mypy:types-PyYAML,types-attrs,attrs,types-requests,types-python-dateutil,[email protected] for python@python3 exceeds tier max size 250MiB: 262.2MiB

So we disabled MyPy checks as part of #485
@tatiana tatiana temporarily deployed to internal September 6, 2023 11:53 — with GitHub Actions Inactive
@tatiana tatiana merged commit 6c3d673 into main Sep 6, 2023
36 of 37 checks passed
@tatiana tatiana deleted the pre-commit-ci-update-config branch September 6, 2023 12:41
tatiana added a commit that referenced this pull request Sep 6, 2023
@tatiana
Enable MyPy checks in the CI, since pre-commit reached quota limit
a0f7553 
As observed in:
#482

The following exception was raised:
build of https://github.com/pre-commit/mirrors-mypy:types-PyYAML,types-attrs,attrs,types-requests,types-python-dateutil,[email protected] for python@python3 exceeds tier max size 250MiB: 262.2MiB

So we disabled MyPy checks as part of #485
tatiana added a commit that referenced this pull request Sep 6, 2023
@tatiana
Enable type checks (MyPy) in the CI (#510)
6c3c003 
We had to disable MyPy checks from the pre-commit hook because we were
reaching a quota from the pre-commit service:
```
build of https://github.com/pre-commit/mirrors-mypy:types-PyYAML,types-attrs,attrs,types-requests,types-python-dateutil,[email protected] for python@python3 exceeds tier max size 250MiB: 262.2MiB
```

As observed in #482.

This PR re-enables MyPy checks but in the CI.

Bonus: Fix Openlineage import error handling
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tatiana tatiana tatiana approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@tatiana