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.

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

fix: don't depend on the GitHub API to check release #391

Merged
merged 2 commits into from Jan 30, 2023

Conversation

crazy-max
Copy link
Member

follow-up docker/buildx#1563

We are currently using the GitHub API to check for latest and tagged releases to make sure they exist before download. But this requires using a token to avoid rate-limit. It's fine for public runners but GHES runners don't have the github.token populated automatically (see docker/setup-buildx-action#194). They need to create a PAT.

We actually don't need to use the GitHub API at all as we have the list of releases available:

"@actions/http-client": "^2.0.1",
"@actions/tool-cache": "^2.0.1",
"js-yaml": "^4.1.0",
"semver": "^7.3.8",
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also found that semver module was not properly registered.

@@ -23,10 +23,10 @@
"dependencies": {
"@actions/core": "^1.10.0",
"@actions/exec": "^1.1.1",
"@actions/github": "^5.1.1",
Copy link
Member Author

@crazy-max crazy-max Jan 29, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This module is not necessary anymore

@codecov
Copy link

codecov bot commented Jan 29, 2023

Codecov Report

Merging #391 (7315d07) into master (9754a25) will increase coverage by 2.23%.
The diff coverage is 83.87%.

@@            Coverage Diff             @@
##           master     #391      +/-   ##
==========================================
+ Coverage   64.95%   67.18%   +2.23%     
==========================================
  Files           3        3              
  Lines         117      128      +11     
  Branches       18       22       +4     
==========================================
+ Hits           76       86      +10     
+ Misses         28       27       -1     
- Partials       13       15       +2     
Impacted Files Coverage Δ
src/context.ts 83.33% <ø> (ø)
src/github.ts 81.13% <82.75%> (-1.37%) ⬇️
src/goreleaser.ts 55.07% <100.00%> (+1.55%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Copy link
Member

@caarlos0 caarlos0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do you think this should be a v4 release?

@crazy-max
Copy link
Member Author

@caarlos0 I don't think it's necessary. Introducing github_token should have been a v4 but we are going back to what we had before so I think it's fine to create a minor release.

@caarlos0
Copy link
Member

@caarlos0 I don't think it's necessary. Introducing github_token should have been a v4 but we are going back to what we had before so I think it's fine to create a minor release.

ah, right

lgtm then :D

@crazy-max
Copy link
Member Author

Oh sorry I misunderstood, I thought you were saying a new major (v5) so yeah we shoud do a release

@crazy-max
Copy link
Member Author

v4.2.0 and v4

@crazy-max crazy-max merged commit f82d6c1 into goreleaser:master Jan 30, 2023
@crazy-max crazy-max deleted the dl-no-token branch January 30, 2023 13:11
@caarlos0
Copy link
Member

Oh sorry I misunderstood, I thought you were saying a new major (v5) so yeah we shoud do a release

I was, but misstyped 😂

all good, your argument makes 100% sense and this seems like the way to go!

renovate bot added a commit to DelineaXPM/terraform-provider-dsv that referenced this pull request Jan 23, 2024
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [actions/checkout](https://togithub.com/actions/checkout) | action |
major | `v3` -> `v4` |
|
[crazy-max/ghaction-import-gpg](https://togithub.com/crazy-max/ghaction-import-gpg)
| action | major | `v5` -> `v6` |
|
[goreleaser/goreleaser-action](https://togithub.com/goreleaser/goreleaser-action)
| action | major | `v3.2.0` -> `v5.0.0` |
| [magefile/mage-action](https://togithub.com/magefile/mage-action) |
action | major | `v2` -> `v3` |
|
[magnetikonline/action-golang-cache](https://togithub.com/magnetikonline/action-golang-cache)
| action | major | `v3` -> `v4` |

---

### Release Notes

<details>
<summary>actions/checkout (actions/checkout)</summary>

###
[`v4`](https://togithub.com/actions/checkout/blob/HEAD/CHANGELOG.md#v400)

[Compare Source](https://togithub.com/actions/checkout/compare/v3...v4)

- [Support fetching without the --progress
option](https://togithub.com/actions/checkout/pull/1067)
-   [Update to node20](https://togithub.com/actions/checkout/pull/1436)

</details>

<details>
<summary>crazy-max/ghaction-import-gpg
(crazy-max/ghaction-import-gpg)</summary>

###
[`v6`](https://togithub.com/crazy-max/ghaction-import-gpg/compare/v5...v6)

[Compare
Source](https://togithub.com/crazy-max/ghaction-import-gpg/compare/v5...v6)

</details>

<details>
<summary>goreleaser/goreleaser-action
(goreleaser/goreleaser-action)</summary>

###
[`v5.0.0`](https://togithub.com/goreleaser/goreleaser-action/releases/tag/v5.0.0)

[Compare
Source](https://togithub.com/goreleaser/goreleaser-action/compare/v4.6.0...v5.0.0)

#### What's Changed

- feat: node 20 as default runtime (requires [Actions Runner
v2.308.0](https://togithub.com/actions/runner/releases/tag/v2.308.0) or
later) by [@&#8203;crazy-max](https://togithub.com/crazy-max) in
[goreleaser/goreleaser-action#432
- chore(deps): bump
[@&#8203;actions/core](https://togithub.com/actions/core) from 1.10.0 to
1.10.1 in
[goreleaser/goreleaser-action#434

**Full Changelog**:
goreleaser/goreleaser-action@v4.6.0...v5.0.0

###
[`v4.6.0`](https://togithub.com/goreleaser/goreleaser-action/releases/tag/v4.6.0)

[Compare
Source](https://togithub.com/goreleaser/goreleaser-action/compare/v4.5.0...v4.6.0)

Reverts the change to `node20` runtime.

**Full Changelog**:
goreleaser/goreleaser-action@v4.5.0...v4.6.0

###
[`v4.5.0`](https://togithub.com/goreleaser/goreleaser-action/releases/tag/v4.5.0)

[Compare
Source](https://togithub.com/goreleaser/goreleaser-action/compare/v4.4.0...v4.5.0)

#### What's Changed

- chore(deps): bump word-wrap from 1.2.3 to 1.2.5 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[goreleaser/goreleaser-action#427
- feat: bump to use node20 runtime, actions/checkout to v4 by
[@&#8203;kbdharun](https://togithub.com/kbdharun) in
[goreleaser/goreleaser-action#430

#### New Contributors

- [@&#8203;kbdharun](https://togithub.com/kbdharun) made their first
contribution in
[goreleaser/goreleaser-action#430

**Full Changelog**:
goreleaser/goreleaser-action@v4.4.0...v4.5.0

###
[`v4.4.0`](https://togithub.com/goreleaser/goreleaser-action/releases/tag/v4.4.0)

[Compare
Source](https://togithub.com/goreleaser/goreleaser-action/compare/v4.3.0...v4.4.0)

#### What's Changed

- chore(deps): bump semver from 7.5.0 to 7.5.1 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[goreleaser/goreleaser-action#417
- feat: support oss nightlies by
[@&#8203;caarlos0](https://togithub.com/caarlos0) in
[goreleaser/goreleaser-action#424

**Full Changelog**:
goreleaser/goreleaser-action@v4.3.0...v4.4.0

###
[`v4.3.0`](https://togithub.com/goreleaser/goreleaser-action/releases/tag/v4.3.0)

[Compare
Source](https://togithub.com/goreleaser/goreleaser-action/compare/v4.2.0...v4.3.0)

#### What's Changed

- Update in command examples by
[@&#8203;arnaduga](https://togithub.com/arnaduga) in
[goreleaser/goreleaser-action#393
- chore(deps): bump yargs from 17.6.2 to 17.7.0 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[goreleaser/goreleaser-action#395
- Improve documentation for use of `GITHUB_TOKEN` by
[@&#8203;jamietanna](https://togithub.com/jamietanna) in
[goreleaser/goreleaser-action#399
- chore(deps): bump actions/setup-go from 3 to 4 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[goreleaser/goreleaser-action#403
- chore(deps): bump docker/bake-action from 2 to 3 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[goreleaser/goreleaser-action#408
- chore(deps): bump semver from 7.3.8 to 7.5.0 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[goreleaser/goreleaser-action#407
- Bump setup-go action version to v4 in README by
[@&#8203;kishaningithub](https://togithub.com/kishaningithub) in
[goreleaser/goreleaser-action#411
- ci: split test and validate workflow by
[@&#8203;crazy-max](https://togithub.com/crazy-max) in
[goreleaser/goreleaser-action#413
- chore: update yarn to 3.5.1 by
[@&#8203;crazy-max](https://togithub.com/crazy-max) in
[goreleaser/goreleaser-action#412
- chore(deps): bump yargs from 17.7.1 to 17.7.2 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[goreleaser/goreleaser-action#410
- feat: support nightly by
[@&#8203;caarlos0](https://togithub.com/caarlos0) in
[goreleaser/goreleaser-action#419

#### New Contributors

- [@&#8203;arnaduga](https://togithub.com/arnaduga) made their first
contribution in
[goreleaser/goreleaser-action#393
- [@&#8203;jamietanna](https://togithub.com/jamietanna) made their first
contribution in
[goreleaser/goreleaser-action#399
- [@&#8203;kishaningithub](https://togithub.com/kishaningithub) made
their first contribution in
[goreleaser/goreleaser-action#411

**Full Changelog**:
goreleaser/goreleaser-action@v4.2.0...v4.3.0

###
[`v4.2.0`](https://togithub.com/goreleaser/goreleaser-action/releases/tag/v4.2.0)

[Compare
Source](https://togithub.com/goreleaser/goreleaser-action/compare/v4.1.1...v4.2.0)

##### What's Changed

- fix: don't depend on the GitHub API to check release by
[@&#8203;crazy-max](https://togithub.com/crazy-max) in
[goreleaser/goreleaser-action#391

**Full Changelog**:
goreleaser/goreleaser-action@v4.1.1...v4.2.0

###
[`v4.1.1`](https://togithub.com/goreleaser/goreleaser-action/releases/tag/v4.1.1)

[Compare
Source](https://togithub.com/goreleaser/goreleaser-action/compare/v4.1.0...v4.1.1)

##### What's Changed

- Update Readme to reference `goreleaser/goreleaser-action@v4` by
[@&#8203;felladrin](https://togithub.com/felladrin) in
[goreleaser/goreleaser-action#384
- docs: fix README badge by
[@&#8203;dirien](https://togithub.com/dirien) in
[goreleaser/goreleaser-action#386
- chore(deps): bump json5 from 2.2.0 to 2.2.3 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[goreleaser/goreleaser-action#388
- fix: use [@&#8203;action/github](https://togithub.com/action/github)
by [@&#8203;caarlos0](https://togithub.com/caarlos0) and
[@&#8203;crazy-max](https://togithub.com/crazy-max) in
[goreleaser/goreleaser-action#390

##### New Contributors

- [@&#8203;felladrin](https://togithub.com/felladrin) made their first
contribution in
[goreleaser/goreleaser-action#384
- [@&#8203;dirien](https://togithub.com/dirien) made their first
contribution in
[goreleaser/goreleaser-action#386

**Full Changelog**:
goreleaser/goreleaser-action@v4.1.0...v4.1.1

###
[`v4.1.0`](https://togithub.com/goreleaser/goreleaser-action/releases/tag/v4.1.0)

[Compare
Source](https://togithub.com/goreleaser/goreleaser-action/compare/v4.0.0...v4.1.0)

##### What's Changed

- feat!: remove auto-snapshot on dirty tag by
[@&#8203;caarlos0](https://togithub.com/caarlos0) in
[goreleaser/goreleaser-action#382
- docs: add example when using workdir along with upload-artifact by
[@&#8203;zdtsw](https://togithub.com/zdtsw) in
[goreleaser/goreleaser-action#366
- Fix Self-Hosted Windows Error: Expand-Archive by
[@&#8203;flarco](https://togithub.com/flarco) in
[goreleaser/goreleaser-action#379
- chore(deps): bump minimatch from 3.0.4 to 3.1.2 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[goreleaser/goreleaser-action#383

##### Migrating from v3

If you need the auto-snapshot feature, take a look at [this example
repository](https://togithub.com/caarlos0/goreleaser-action-v4-auto-snapshot-example):
it's a minimal working example with all you need.

##### New Contributors

- [@&#8203;zdtsw](https://togithub.com/zdtsw) made their first
contribution in
[goreleaser/goreleaser-action#366
- [@&#8203;flarco](https://togithub.com/flarco) made their first
contribution in
[goreleaser/goreleaser-action#379

**Full Changelog**:
goreleaser/goreleaser-action@v4...v4.1.0

###
[`v4.0.0`](https://togithub.com/goreleaser/goreleaser-action/releases/tag/v4.0.0)

[Compare
Source](https://togithub.com/goreleaser/goreleaser-action/compare/v3.2.0...v4.0.0)

##### What's Changed

- fix: do not override GORELEASER_CURRENT_TAG by
[@&#8203;caarlos0](https://togithub.com/caarlos0) in
[goreleaser/goreleaser-action#370

##### Migrating from v3

If you need the auto-snapshot feature, take a look at [this example
repository](https://togithub.com/caarlos0/goreleaser-action-v4-auto-snapshot-example):
it's a minimal working example with all you need.

**Full Changelog**:
goreleaser/goreleaser-action@v3...v4.0.0

</details>

<details>
<summary>magefile/mage-action (magefile/mage-action)</summary>

### [`v3`](https://togithub.com/magefile/mage-action/compare/v2...v3)

[Compare
Source](https://togithub.com/magefile/mage-action/compare/v2...v3)

</details>

<details>
<summary>magnetikonline/action-golang-cache
(magnetikonline/action-golang-cache)</summary>

###
[`v4`](https://togithub.com/magnetikonline/action-golang-cache/releases/tag/v4)

[Compare
Source](https://togithub.com/magnetikonline/action-golang-cache/compare/v3...v4)

- Provide action outputs of `build-cache-path`, `module-cache-path` and
`cache-key`.
-   Bump `actions/setup-go@v4`
-   Some `README.md` cleanups.

</details>

---

### Configuration

📅 **Schedule**: Branch creation - "every weekday" (UTC), Automerge - At
any time (no schedule defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

👻 **Immortal**: This PR will be recreated if closed unmerged. Get
[config help](https://togithub.com/renovatebot/renovate/discussions) if
that's undesired.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/DelineaXPM/terraform-provider-dsv).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4xMzUuMCIsInVwZGF0ZWRJblZlciI6IjM3LjEzNS4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiJ9-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants