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.

Sign up for GitHub

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

Jump to bottom

Issue detecting commit SHA #190

Closed
johnwheal opened this issue Dec 29, 2020 · 4 comments
Closed

Issue detecting commit SHA #190

johnwheal opened this issue Dec 29, 2020 · 4 comments

Comments

@johnwheal
Copy link

Since release v1.1.0 I am getting the following error:

-> Issue detecting commit SHA. Please run actions/checkout with fetch-depth > 1 or set to 0
@ktomk
Copy link

ktomk commented Dec 30, 2020

Already triaged in codecov-bash codecov/codecov-bash#397 , a new action release with this revision would be nice, on Github I see Bash-20201223-5ad85e3 since days and could not yet confirm good.

@thomasrockhu
Copy link
Contributor

This has been released. If you are still experiencing the issue, you will likely need to set fetch_depth != 1 in your checkout stage.

@ktomk
Copy link

ktomk commented Jan 3, 2021

I can now confirm:

old: Bash-20201223-5ad85e3
fix: Bash-20201231-23d4836

Thanks @thomasrockhu

@brendankenny brendankenny mentioned this issue Feb 10, 2021
Merged
@vytas7 vytas7 mentioned this issue Feb 14, 2021
Merged
@ste93cry ste93cry mentioned this issue Feb 19, 2021
Merged
prince-chrismc added a commit to prince-chrismc/user-management that referenced this issue Feb 20, 2021
@prince-chrismc
Update node.js.yml
94d6f13 
codecov/codecov-action#190
prince-chrismc added a commit to prince-chrismc/user-management that referenced this issue Feb 21, 2021
@prince-chrismc
🚧 🧱 Transition front-end to use TypeScript and Sass (#100)
8265e92 
* working poc using typescript

* clean up tsconfig

* rm extra

* switch some componetents to `.tsx`

* upgrade to sass

* rename files

* rework webpack config files

* fix mocking fetch

* more linting

* fix prop-types

without import typescript complains

* moving base import to setup

* remove jsx attribution from core functions

* add a core model + props interface for select message

* enable compile type checking

* fixing some errors

* zap since it's completely changed

* playing with prop-types and typescript

* comment out proptypes

TBA on what the answer will be here

* move layout to containers

* 🐛 found by typescript

* yarn upgrade --latest

* Update node.js.yml

codecov/codecov-action#190

* fix coverage files includes

* Delete .eslintignore

* Update layout.scss.d.ts

* Update tsconfig.json
li-boxuan added a commit to li-boxuan/janusgraph that referenced this issue Feb 24, 2021
@li-boxuan
Fix codecov configs
0aa3e4c 
1. Notify config should be under codecov group, not coverage group
2. Remove codecov.notify.after_n_builds limit so that code coverage
is always available as a GitHub check, no matter how many codecov
reports are uploaded
3. Disable pull request comment feature since it is annoying and
unstable
4. Name codecov reports
5. Turn on patch status check
6. Add fetch-depth config to fix issue reported at
codecov/codecov-action#190

Signed-off-by: Boxuan Li <[email protected]>
li-boxuan added a commit to JanusGraph/janusgraph that referenced this issue Feb 25, 2021
@li-boxuan
Fix codecov configs
c493377 
1. Notify config should be under codecov group, not coverage group
2. Remove codecov.notify.after_n_builds limit so that code coverage
is always available as a GitHub check, no matter how many codecov
reports are uploaded
3. Disable pull request comment feature since it is annoying and
unstable
4. Name codecov reports
5. Turn on patch status check
6. Add fetch-depth config to fix issue reported at
codecov/codecov-action#190

Signed-off-by: Boxuan Li <[email protected]>
@viceice viceice mentioned this issue Mar 2, 2021
Merged
6 tasks
@Yoshanuikabundi Yoshanuikabundi mentioned this issue Mar 3, 2021
Merged
llucax added a commit to noclick-me/noclick.me that referenced this issue Mar 3, 2021
@llucax
ci: Increase checkout depth to 2
fbe6f56 
This is needed by codecov to properly detect the commit. See
codecov/codecov-action#190 (comment).

Signed-off-by: Leandro Lucarella <[email protected]>
Jean85 added a commit to facile-it/terminable-loop-command that referenced this issue Mar 4, 2021
@Jean85
Increase the fetch size in CI to 2
18b80f7 
This is needed by codecov to properly detect the commit in PRs. See codecov/codecov-action#190 (comment)
@Jean85 Jean85 mentioned this issue Mar 4, 2021
Closed
@silva96
Copy link

silva96 commented Mar 4, 2021 •
edited
Loading

if you are using github actions with the codecov bash uploader

- uses: actions/checkout@v2
  with:
    fetch-depth: 2

gicmo added a commit to gicmo/osbuild that referenced this issue Mar 11, 2021
@gicmo
ci: increase fetch-depth to 2
c6b0b91 
Codecov apparently has trouble detecting the git commit if the
fetch depth is only 1 (the default). Therefore increase it.
See codecov/codecov-action#190 for
more details.
teg pushed a commit to osbuild/osbuild that referenced this issue Mar 11, 2021
@gicmo @teg
ci: increase fetch-depth to 2
99160ad 
Codecov apparently has trouble detecting the git commit if the
fetch depth is only 1 (the default). Therefore increase it.
See codecov/codecov-action#190 for
more details.
@dhiltgen dhiltgen mentioned this issue Mar 23, 2021
Merged
@Flamefire Flamefire mentioned this issue Apr 8, 2021
Open
@Irallia Irallia mentioned this issue Apr 30, 2021
Merged
@nguyen102 nguyen102 mentioned this issue May 6, 2021
Merged
2 tasks
christianlupus added a commit to nextcloud/cookbook that referenced this issue Jul 2, 2021
@christianlupus
Correct fetching depth in action scripts
26195e9 
See codecov/codecov-action#190 (comment)

Signed-off-by: Christian Wolf <[email protected]>
@christianlupus christianlupus mentioned this issue Jul 2, 2021
Merged
@anmyachev anmyachev mentioned this issue Jul 12, 2021
Closed
Praveenrajmani added a commit to Praveenrajmani/directpv that referenced this issue Jul 22, 2021
@Praveenrajmani
fix github actions setting
755068c 
```
issue detecting commit SHA. Please run actions/checkout with fetch-depth > 1 or set to 0
```

Reference: codecov/codecov-action#190
wlan0 pushed a commit to minio/directpv that referenced this issue Jul 23, 2021
@Praveenrajmani @wlan0
fix github actions setting
bb8eb54 
```
issue detecting commit SHA. Please run actions/checkout with fetch-depth > 1 or set to 0
```

Reference: codecov/codecov-action#190
@kennethshackleton kennethshackleton mentioned this issue Aug 29, 2021
Merged
arl added a commit to arl/statsviz that referenced this issue Sep 4, 2021
@arl
Update coverage.yml
cbb0905 
See codecov/codecov-action#190
arl added a commit to arl/statsviz that referenced this issue Sep 4, 2021
@arl
Embed static assets (#55)
3c3c962 
* Remove assets-related Go files

* Move static -> internal/static

* go.mod: min go1.16 for embed package

* Use go:embedded assets

* _example/options: print handler path

* .github: remove Assets workflow

The Assets workflow is not needed anymore since assets are always
up to date thanks to the usage of the embed package.

* README.md: add Go version section

* Unexport Assets

* CONTRIBUTING.md: remove reference to assets generation

* .codecov.yml: remove exception for assets_*.go

* Update CHANGELOG.md

* Update coverage.yml

See codecov/codecov-action#190
@eivindjahren eivindjahren mentioned this issue Oct 6, 2021
Merged
rqbacktrace added a commit to backtrace-labs/backtrace-android that referenced this issue Jan 7, 2022
@rqbacktrace
Set fetch-depth to 2 for codecov
2bd142f 
codecov/codecov-action#190 (comment)
ghiscoding added a commit to ghiscoding/slickgrid-universal that referenced this issue May 20, 2022
@ghiscoding
ci: increase fetch-depth to 2
96968fe 
-Codecov apparently has trouble detecting the git commit if the
fetch depth is only 1 (the default). Therefore increase it.
See codecov/codecov-action#190 for
more details.
This was referenced Jun 30, 2022
Merged
Merged
@nicolaslg nicolaslg mentioned this issue Aug 23, 2022
Closed
fasterthanlime added a commit to bearcove/loona that referenced this issue Nov 2, 2022
@fasterthanlime
Try fetch-depth: 2 to appease codecov
eefa2b5 
cf. codecov/codecov-action#190 (comment)
@wrandall22 wrandall22 mentioned this issue Jul 20, 2023
Merged
Wxh16144 added a commit to Wxh16144-forks/util that referenced this issue Apr 29, 2024
@Wxh16144
ci: fixed codecov warning
86ffdfc 
ref: codecov/codecov-action#190
@Wxh16144 Wxh16144 mentioned this issue Apr 29, 2024
Merged
li-jia-nan pushed a commit to react-component/util that referenced this issue Apr 29, 2024
@Wxh16144
ci: fixed codecov warning (#526)
2cdbaf3 
ref: codecov/codecov-action#190
li-jia-nan added a commit to react-component/util that referenced this issue Apr 29, 2024
@li-jia-nan @Wxh16144
test: rm legacy react-dom/test-utils (#525)
63fc17e 
* test: rm legacy react-dom/test-utils

* chore: fix

* ci: fixed codecov warning (#526)

ref: codecov/codecov-action#190

---------

Co-authored-by: 红 <[email protected]>
kinow added a commit to kinow/protobuf-uml-diagram that referenced this issue Apr 29, 2024
@kinow
Use fetch-depth of 2 for codecov issue - codecov/codecov-action#190 (…
d3706b1 
…comment)
@gtdang gtdang mentioned this issue Apr 30, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@ktomk @johnwheal @silva96 @thomasrockhu