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 typos #739

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fix typos #739

wants to merge 1 commit into from

Conversation

Gelma
Copy link
Contributor

@Gelma Gelma commented Sep 1, 2024

Hi guys, thanks a lot for your work.

Here's a stupid patch about stupid detalis.

Thanks a lot,
Gelma

@koverstreet
Copy link
Owner

this should all be one commit

@Gelma
Copy link
Contributor Author

Gelma commented Sep 1, 2024

this should all be one commit

Yeap, the idea is to have it each one just for quick review, and to squash it before merge (GitHub interface lets you do it before merge). Anyway, I update in one squash!

@Gelma
Copy link
Contributor Author

Gelma commented Sep 2, 2024

Rebased the patch on latest commit.

@Gelma
Copy link
Contributor Author

Gelma commented Sep 2, 2024

New update. Added fix for: "reentrency", "durabilty", "definites", "ascestor"

@Gelma
Copy link
Contributor Author

Gelma commented Sep 4, 2024

Updated with latest commits.

@Gelma
Copy link
Contributor Author

Gelma commented Sep 4, 2024

Added a new fix for "sentinal"

@Gelma
Copy link
Contributor Author

Gelma commented Sep 5, 2024

Updated to latest commits

@koverstreet koverstreet force-pushed the master branch 4 times, most recently from 7d754ac to 173f06e Compare November 29, 2024 04:21
@koverstreet koverstreet force-pushed the master branch 2 times, most recently from a7b3ff7 to 4a32728 Compare December 4, 2024 02:50
@koverstreet koverstreet force-pushed the master branch 9 times, most recently from 23dec8a to dd7d7f2 Compare December 9, 2024 13:52
@koverstreet koverstreet force-pushed the master branch 2 times, most recently from 79c1539 to f8ed207 Compare December 12, 2024 07:29
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.

3 participants