Skip to content

Conversation

Sergix
Copy link
Owner

@Sergix Sergix commented Jun 12, 2017

Time for a patch!

This corrects some issues with JTerm crashing from invalid input, and adds some licensing information into the program. Check out the associated Patch Notes for more details.

There is also now a 'pom.xml' in the project, but Travis CI won't run the Maven script I'm telling it to in the yml... ignore Travis CI if it fails for this build. I've already run the necessary checks myself.

Please review and merge if all is well, @NCSGeek ! 📦 🚀

Sergix added 7 commits June 10, 2017 17:32
Resolves conflict that was fixed before v0.3.0.
View changelog for details.
View changelog for details.
View changelog for details.
View changelog and patch notes for details.
@Sergix Sergix added the bug label Jun 12, 2017
@Sergix Sergix requested a review from NCSGeek June 12, 2017 19:31
@Sergix
Copy link
Owner Author

Sergix commented Jun 12, 2017

Nevermind about the Travis CI thing. Finally configured the POM to work!
Yay for Maven integration. 👍

@NCSGeek
Copy link
Collaborator

NCSGeek commented Jun 12, 2017

Looks Good!

@NCSGeek NCSGeek merged commit a7c38ff into master Jun 12, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants