Skip to content

Make shadowJar less onerous#475

Merged
nwbirnie merged 4 commits intomasterfrom
onerousshadow
Aug 27, 2021
Merged

Make shadowJar less onerous#475
nwbirnie merged 4 commits intomasterfrom
onerousshadow

Conversation

@nwbirnie
Copy link
Copy Markdown
Contributor

A couple of improvements to build velocity here:

  • use the output files of the generate licenses task. This means that shadowJar can use the cache (can remove upToDateWhen=false).
  • SmokeTest is only blocking on publish to sonatype.
  • SmokeTest times out after 20 seconds (e.g. in case of bad OAuth credentials).
  • check for errors in SmokeTest (this was a bug).

Change-Id: I27970e69eec8af653a2925a34220a40a929d5db1

A couple of improvements to build velocity here:
- use the output files of the generate licenses task. This means that shadowJar can use the cache (can remove upToDateWhen=false).
- SmokeTest is only blocking on publish to sonatype.
- SmokeTest times out after 20 seconds (e.g. in case of bad OAuth credentials).
- check for errors in SmokeTest (this was a bug).

Change-Id: I27970e69eec8af653a2925a34220a40a929d5db1
@nwbirnie nwbirnie requested review from devchas and jradcliff August 26, 2021 13:59
Comment thread google-ads/src/test/functional/java/com/google/ads/googleads/lib/SmokeTest.java Outdated
Comment thread google-ads/build.gradle
Comment thread google-ads/build.gradle
Change-Id: I520ee363998660c5415cf852c6df8ed722e27d44
Change-Id: Ibf8a241803dcc5047974686278d76252aa5cdec2
@nwbirnie nwbirnie requested a review from jradcliff August 27, 2021 12:57
Comment thread google-ads/build.gradle
Change-Id: I1b5e2d7a8e91a27c1b73848bb3830b4d29a20069
@nwbirnie nwbirnie requested a review from jradcliff August 27, 2021 14:50
@nwbirnie nwbirnie merged commit 0c3c22d into master Aug 27, 2021
@nwbirnie nwbirnie deleted the onerousshadow branch August 27, 2021 16:49
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