Skip to content

Speed up builds #267

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 5 commits into from
Jan 21, 2023
Merged

Speed up builds #267

merged 5 commits into from
Jan 21, 2023

Conversation

@ascopes
Copy link
Owner

@ascopes ascopes commented Jan 21, 2023 •
edited
Loading

  • Speed up builds by uploading one bulk report tarball rather than individual files.
  • Switch out GZIP with XZ and use the maximum compression factor.

@ascopes ascopes added the ci Changes to CI workflows label Jan 21, 2023
@ascopes ascopes self-assigned this Jan 21, 2023
@ascopes ascopes changed the title Speed up builds by uploading report tarball rather than individual fi… Speed up builds by uploading report tarball rather than individual files Jan 21, 2023
@ascopes ascopes changed the title Speed up builds by uploading report tarball rather than individual files Speed up builds Jan 21, 2023
@ascopes ascopes force-pushed the task/speed-up-builds branch from 6b2a1df to fb5068a Compare January 21, 2023 14:12
@github-actions
Copy link

github-actions bot commented Jan 21, 2023 •
edited
Loading

Test Results

  1 139 files  +10    1 139 suites  +10   0s ⏱️ ±0s
14 798 tests +90  14 788 ✔️ +80  10 💤 +10  0 ❌ ±0 
15 028 runs  +90  15 018 ✔️ +80  10 💤 +10  0 ❌ ±0 

Results for commit 99b82d0. ± Comparison against base commit 902f0c5.

♻️ This comment has been updated with latest results.

@codecov-commenter
Copy link

codecov-commenter commented Jan 21, 2023 •
edited
Loading

Codecov Report

Merging #267 (99b82d0) into main (62f7915) will increase coverage by 0.04%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #267      +/-   ##
==========================================
+ Coverage   71.12%   71.17%   +0.04%     
==========================================
  Files          73       74       +1     
  Lines        2556     2560       +4     
  Branches      239      238       -1     
==========================================
+ Hits         1818     1822       +4     
  Misses        671      671              
  Partials       67       67
Impacted Files Coverage Δ
...opes/jct/filemanagers/LoggingFileManagerProxy.java 1.58% <0.00%> (ø)
...pes/jct/diagnostics/TracingDiagnosticListener.java 100.00% <0.00%> (ø)
...ners/impl/PackageContainerGroupUrlClassLoader.java 100.00% <0.00%> (ø)
...java/io/github/ascopes/jct/utils/LoomPolyfill.java 100.00% <0.00%> (ø)

@ascopes ascopes force-pushed the task/speed-up-builds branch 3 times, most recently from 64e50d8 to e069ba5 Compare January 21, 2023 15:02
@ascopes
Speed up builds by uploading report tarball rather than individual fi…
52e361a 
…les at the end.

Switch out GZIP with XZ and use the maximum compression factor.

Provide Maven default flags for JVM and build config
@ascopes ascopes force-pushed the task/speed-up-builds branch from e069ba5 to feb33a2 Compare January 21, 2023 15:29
@ascopes ascopes force-pushed the task/speed-up-builds branch from feb33a2 to 761bd0a Compare January 21, 2023 15:40
@ascopes ascopes merged commit 30af4f9 into main Jan 21, 2023
@ascopes ascopes deleted the task/speed-up-builds branch January 21, 2023 17:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@ascopes ascopes

Labels

ci Changes to CI workflows

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ascopes @codecov-commenter