Skip to content

Large dependency fast-forward - #191

Draft
CocoisBuggy wants to merge 2 commits into
developfrom
coco-actions
Draft

CocoisBuggy wants to merge 2 commits into
developfrom
coco-actions

Conversation

@CocoisBuggy

Copy link
Copy Markdown
Contributor

There's a lot to dig into here but I thought I would explore just how much of the project explodes if we try and bring it forward a bit.

I initially just wanted to touch the github actions to use the newer secure versions (v3 -> v7 for the checkout action) and found that the node version was pinned too low to allow this. Actually fast forwarding required that a bunch of the toolchain come along for the ride, hence the large diff.

I would hold off on merging this for the time being, leave the PR as a draft for a bit

@github-actions

Copy link
Copy Markdown

Coverage report

Caution

Coverage does not meet threshold
Statements coverage not met for global: expected <=999999 not covered statements, but got 635

St.
Category Percentage Covered / Total
🟢 Statements 87.95% 635/722
🟡 Branches 67.54% 129/191
🟡 Functions 79.27% 130/164
🟢 Lines 86.89% 570/656

Test suite run success

268 tests passing in 17 suites.

Report generated by 🧪jest coverage report action from 6bc91c0

This branch has not been deployed

No deployments
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.

1 participant