bahmutov / bahmutov/changed-cy-tests

Unnecessary checkout

Open
#4 1 comment 0 reactions 0 assignees View on GitHub
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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.