You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@pekko.apache.org by "mdedetrich (via GitHub)" <gi...@apache.org> on 2023/09/05 09:48:49 UTC
[GitHub] [incubator-pekko-sbt-paradox] mdedetrich opened a new pull request, #78: Use fetch tags from latest checkout action
mdedetrich opened a new pull request, #78:
URL: https://github.com/apache/incubator-pekko-sbt-paradox/pull/78
The latest version of the checkout github actions now supports a `fetch-tags` flag which does as described, i.e. it will fetch all of our tags which removes the various workarounds that was used before (i.e. manually running `git fetch` command) etc etc, see https://github.com/actions/checkout/pull/579
This will also help in the erroneous errors that we get in CI, i.e.
```
[error] Failed to derive version from git tags. Maybe run `git fetch --unshallow` or `git fetch upstream` on a fresh git clone from a fork? Derived version: 0.0.0+1-337943bd-SNAPSHOT`
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@pekko.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@pekko.apache.org
For additional commands, e-mail: notifications-help@pekko.apache.org
[GitHub] [incubator-pekko-sbt-paradox] mdedetrich merged pull request #78: Use fetch tags from latest checkout action
Posted by "mdedetrich (via GitHub)" <gi...@apache.org>.
mdedetrich merged PR #78:
URL: https://github.com/apache/incubator-pekko-sbt-paradox/pull/78
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@pekko.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@pekko.apache.org
For additional commands, e-mail: notifications-help@pekko.apache.org
[GitHub] [incubator-pekko-sbt-paradox] mdedetrich commented on a diff in pull request #78: Use fetch tags from latest checkout action
Posted by "mdedetrich (via GitHub)" <gi...@apache.org>.
mdedetrich commented on code in PR #78:
URL: https://github.com/apache/incubator-pekko-sbt-paradox/pull/78#discussion_r1315659959
##########
.github/workflows/publish.yml:
##########
@@ -11,9 +11,10 @@ jobs:
if: github.repository == 'apache/incubator-pekko-sbt-paradox'
steps:
- name: Checkout
- uses: actions/checkout@v3
+ uses: actions/checkout@v4
with:
- fetch-depth: 100
Review Comment:
Using a `fetch-depth` of 100 here would be wrong because the published version would end up being incorrect once the git history would get big enough (to the point where it wouldn't fetch the latest tag)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@pekko.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@pekko.apache.org
For additional commands, e-mail: notifications-help@pekko.apache.org