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.

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

Fix more spelling #252

Merged
merged 1 commit into from
Jan 24, 2025
Merged

Fix more spelling #252

merged 1 commit into from
Jan 24, 2025

Conversation

jbampton
Copy link
Member

No description provided.

@jbampton jbampton self-assigned this Jan 24, 2025
@jbampton
Copy link
Member Author

So there are a whole bunch of pre-commit hooks that can be added in future if needed.

The official hooks are seen here: https://github.com/pre-commit/pre-commit-hooks

And on the website there is a larger list of hooks: https://pre-commit.com/hooks.html

@Pilot-Pirx
Copy link
Member

Could you separate the fixes and the codespell.txt next time?
That makes it easier to cherry-pick for the AOO42X branch.

@Pilot-Pirx Pilot-Pirx merged commit 40ce8f8 into apache:trunk Jan 24, 2025
1 check passed
@Pilot-Pirx
Copy link
Member

Pilot-Pirx commented Jan 24, 2025

Sorry, I just merged and now we have a conflict...

Additionally, Apache GitBox is "dead"!

@jbampton jbampton deleted the fix-spelling branch January 24, 2025 13:58
@jbampton
Copy link
Member Author

I fixed the conflict in the other PR.

Next time I will try the two commits for separation.

Thanks

@Pilot-Pirx
Copy link
Member

Great, feel free to merge the other PR.
I will cherry-pick later when GitBox hopefully is functional again.

asfgit pushed a commit that referenced this pull request Jan 24, 2025
(cherry picked from commit 40ce8f8)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants