🛠 Tooling: Are actions/upload-artifact calls necessary in test*.yml? #1225
Labels
area: tooling
Managing the repository's maintenance
status: accepting prs
Please, send a pull request to resolve this!
type: cleanup
Tech debt or other code/repository cleanups
Milestone
Bug Report Checklist
main
branch of the repository.Overview
Right now, the end-to-end migration test snapshot shows that this repo includes
actions/upload-artifact
in itstest.yml
despite the template not including that call:create-typescript-app/script/__snapshots__/migrate-test-e2e.js.snap
Lines 53 to 71 in c4025fa
I don't think that uploading is necessary. Let's delete the lines in
.github/workflows/test*.yml
.Additional Info
This repo has had a few Codecov changes in the past:
test.yml
codecov uploadingcodecov.yml
so that coverage from the four tests (3 x end-to-end, 1 x unit) get merged together before any notifications are sent#473 is what mentions the code coverage reports:
I've never used that convenience. And codecov also has an online viewer.
The text was updated successfully, but these errors were encountered: