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

chore(ci): dynamic matrix for reduced private fork build time #362

Merged
merged 1 commit into from
Jan 11, 2022

Conversation

jansegre
Copy link
Member

@jansegre jansegre commented Jan 10, 2022

This is what happens on a repo that is not the main repo: jansegre#7

This PR is largely based off https://www.cynkra.com/blog/2020-12-23-dynamic-gha/

Acceptance criteria:

  • On the main repo (HathorNetwork/hathor-core), the build matrix should be the complete build matrix, the same as it currently is;
  • On any other repo, the build matrix should be reduced (I chose Python-3.7 on Ubuntu);

@jansegre jansegre self-assigned this Jan 10, 2022
@jansegre jansegre requested a review from msbrogli as a code owner January 10, 2022 20:10
@jansegre jansegre force-pushed the chore/dynamic-ci-matrix branch from 3e2742f to 2d03b20 Compare January 10, 2022 20:11
@jansegre jansegre requested review from luislhl and pedroferreira1 and removed request for msbrogli January 10, 2022 20:21
@jansegre jansegre force-pushed the chore/dynamic-ci-matrix branch from 2d03b20 to 2123d56 Compare January 10, 2022 20:40
@codecov
Copy link

codecov bot commented Jan 10, 2022

Codecov Report

Merging #362 (2123d56) into dev (b84d5fb) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##              dev     #362   +/-   ##
=======================================
  Coverage   82.68%   82.68%           
=======================================
  Files         164      164           
  Lines       15084    15084           
  Branches     2143     2143           
=======================================
  Hits        12472    12472           
+ Misses       2182     2181    -1     
- Partials      430      431    +1     
Impacted Files Coverage Δ
hathor/pubsub.py 89.79% <0.00%> (-4.09%) ⬇️
hathor/consensus.py 95.66% <0.00%> (-0.18%) ⬇️
hathor/p2p/node_sync.py 83.94% <0.00%> (ø)
hathor/p2p/manager.py 69.77% <0.00%> (+1.11%) ⬆️

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 b84d5fb...2123d56. Read the comment docs.

@jansegre jansegre merged commit fa90b52 into dev Jan 11, 2022
@jansegre jansegre deleted the chore/dynamic-ci-matrix branch January 11, 2022 00:21
@jansegre jansegre mentioned this pull request Jan 21, 2022
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