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 #32

Merged
merged 2 commits into from
Nov 4, 2022
Merged

Fix typos #32

merged 2 commits into from
Nov 4, 2022

Conversation

aphedges
Copy link
Contributor

@aphedges aphedges commented Nov 2, 2022

As a follow-up to #28, I have fixed some more typos in the codebase.

I also updated the JAR version in package.sh so it would build properly.

This is a follow-up to the recent typo-fixing commits. This one uses the
IntelliJ spellchecker to catch misspellings that codespell and the
programmer did not find.
Copy link
Collaborator

@MarcCote MarcCote left a comment

Choose a reason for hiding this comment

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

Thanks a lot. LGTM. @PeterAJansen I'll let check it out before merging.

@PeterAJansen
Copy link
Collaborator

Looks good, most of these typos are in the code comments.

There is also a change to the JAR file version. Can you verify this is accurate @MarcCote ?

@aphedges
Copy link
Contributor Author

aphedges commented Nov 3, 2022

@PeterAJansen, the JAR file version was changed in ce05307. That commit missed changing the build script, so I fixed it here.

@MarcCote
Copy link
Collaborator

MarcCote commented Nov 3, 2022 via email

@MarcCote MarcCote merged commit 1dd16e7 into allenai:main Nov 4, 2022
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