bahmutov / bahmutov/changed-cy-tests
Unnecessary checkout
- Dominant language
- JavaScript
- Stars
- 6
- Forks
- 3
- PR merge metrics
- No merged PRs in 30d
Description
I don't think these lines https://github.com/bahmutov/changed-cy-tests/blob/3905af54164646aa997d21389e8c73a4408bb01a/.github/workflows/ci.yml#L15-L19 are needed, the `git fetch` lines are enough to then be able to do perform the `diff` and to get the changed files which are all that's needed further down.
Happy to raise a PR if there isn't something I'm missing here?
Thanks for these and the accompanying blog post - really really useful!
Contributor guide
No contributing guide indexed for this repository
Research direction
Inspect .github/workflows/ci.yml lines 15-19 and compare the checkout steps with the git fetch commands used before the diff. Remove the unnecessary checkout only if the workflow still obtains the changed files and completes its existing changed-spec and full-test flow.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- git, github-actions
- Domain
- ci-cd, devops
- Issue type
- Refactor
- Difficulty
- 1/5
- Estimated time
- Under an hour
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 35/100