Skip to content

Fixes #2

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

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Fixes #2

wants to merge 5 commits into from

Conversation

uthanda
Copy link
Collaborator

@uthanda uthanda commented Jan 4, 2022

I made a few fixes and have updated the pom.xml file to support direct deployment the GitHub maven repo. I would like to create a 0.3 release so we have a full version that people can use. If you accept this pull request, I will perform a proper maven release on the main branch.

Fixed character encoding error.  The Romanian Ș ș characters got garbled.
Fixed build issues where the jlatex.api module was not added to the parent pom and where Maven command line didn't like the UTF-8 encoded Romanian characters.
Addressed a couple of SonarLint warnings
Added SCM information to the pom.xml file for automatic deployment to GitHub Maven repo.
@uthanda uthanda requested a review from bashterm January 4, 2022 12:58
Further corrected the pom.xml file., adding the mvn repo elements to the URL.
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.

1 participant