Skip to content

Upgrade pre-commits#35033

Merged
potiuk merged 1 commit intoapache:mainfrom
potiuk:update-pre-commit
Oct 18, 2023
Merged

Upgrade pre-commits#35033
potiuk merged 1 commit intoapache:mainfrom
potiuk:update-pre-commit

Conversation

@potiuk
Copy link
Member

@potiuk potiuk commented Oct 18, 2023


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in newsfragments.

Copy link
Member

@hussein-awala hussein-awala left a comment

Choose a reason for hiding this comment

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

The new version of codespell looks much better than the previous one!

@potiuk
Copy link
Member Author

potiuk commented Oct 18, 2023

The new version of codespell looks much better than the previous one!

Indeed. And more troublesome. And it actually detected PostregsSQL which I tried to fix three times in spelling_wordlist before I relized we have one s too many

@potiuk potiuk merged commit b75f9e8 into apache:main Oct 18, 2023
@potiuk potiuk deleted the update-pre-commit branch October 18, 2023 19:28
@ephraimbuddy ephraimbuddy added the type:misc/internal Changelog: Misc changes that should appear in change log label Oct 27, 2023
@ephraimbuddy ephraimbuddy added this to the Airflow 2.7.3 milestone Oct 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:API Airflow's REST/HTTP API area:CLI area:dev-tools area:lineage area:providers area:Scheduler including HA (high availability) scheduler area:webserver Webserver related Issues provider:amazon AWS/Amazon - related issues type:misc/internal Changelog: Misc changes that should appear in change log

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants