Skip to content

chore: update Makefile to move jar during install #124

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

Merged
merged 2 commits into from
Feb 3, 2020
Merged

Conversation

eshanholtz
Copy link
Contributor

No description provided.

@thinkingserious thinkingserious added the status: code review request requesting a community code review or review from Twilio label Feb 3, 2020
@codecov
Copy link

codecov bot commented Feb 3, 2020

Codecov Report

Merging #124 into master will increase coverage by 2.81%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #124      +/-   ##
============================================
+ Coverage     46.47%   49.29%   +2.81%     
- Complexity       36       38       +2     
============================================
  Files             5        5              
  Lines           213      213              
  Branches         16       16              
============================================
+ Hits             99      105       +6     
+ Misses          110      104       -6     
  Partials          4        4
Impacted Files Coverage Δ Complexity Δ
src/main/java/com/sendgrid/Client.java 34.5% <0%> (+4.22%) 12% <0%> (+2%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5dd4c83...bf12724. Read the comment docs.

@eshanholtz eshanholtz merged commit bf12724 into master Feb 3, 2020
@eshanholtz eshanholtz deleted the makefile branch February 3, 2020 19:07
@thinkingserious
Copy link
Contributor

Hello @eshanholtz,

Thanks again for the PR!

We appreciate your contribution and look forward to continued collaboration. Thanks!

Team SendGrid DX

install:
@java -version || (echo "Java is not installed, please install Java >= 7"; exit 1);
mvn clean install -DskipTests=true -Dgpg.skip -B
cp target/java-http-client-$(VERSION).jar java-http-client.jar
Copy link
Contributor

Choose a reason for hiding this comment

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

Line needs to start w/ a tab.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: code review request requesting a community code review or review from Twilio
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants