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

Corrupted zip bug fix #9

Merged
merged 1 commit into from
Apr 21, 2017

Conversation

alayouni
Copy link

  • Added source file name header to fix source submission bug
  • Replaced buggy Files#probeContentType call by Tika#detect
  • Added Tika dependency to pom

… buggy Files#probeContentType call by Tika#detect
@coveralls
Copy link

Coverage Status

Coverage increased (+0.1%) to 71.782% when pulling e0516ce on alayouni:alayouni-source-submission-bug-fix into ff17e90 on Faylixe:master.

@Faylixe
Copy link
Owner

Faylixe commented Apr 15, 2017

Thanks for the pull request ! All the update sound ok to me. I will just merge next thursday, because diffusing artifact on maven, and repackaging CLI tools requires private key that is located on my office computer :).

Nice work !

@alayouni
Copy link
Author

alayouni commented Apr 20, 2017

Any a chance for the new release before round 1B?

@Faylixe
Copy link
Owner

Faylixe commented Apr 20, 2017

Yes sorry i had a busy week :). I will do it tomorrow !

@alayouni
Copy link
Author

ok thanks!

@Faylixe Faylixe merged commit 0888d29 into Faylixe:master Apr 21, 2017
@Faylixe
Copy link
Owner

Faylixe commented Apr 21, 2017

Pull request has been merged, reviewed, and deployed into maven central under 1.4.11 version. Google code jam CLI application has been upgraded to 1.4.1 accordingly (https://github.com/Faylixe/googlecodejam-cli/releases/tag/v1.4.1). I added you in the contributor list in the repository README. Thanks again !

@alayouni alayouni deleted the alayouni-source-submission-bug-fix branch April 21, 2017 10:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants