Skip to content

Update GoReleaser to v0.172.1#3926

Merged
samcoe merged 2 commits intocli:trunkfrom
chemotaxis:update-goreleaser-20210630
Jul 15, 2021
Merged

Update GoReleaser to v0.172.1#3926
samcoe merged 2 commits intocli:trunkfrom
chemotaxis:update-goreleaser-20210630

Conversation

@chemotaxis
Copy link
Contributor

@chemotaxis chemotaxis commented Jul 1, 2021

Fixes #3909.

nfpms.files is deprecated: https://goreleaser.com/deprecations/#nfpmsfiles

Tested with this version:

goreleaser version 0.172.1
commit: 32a44ab928879bb32c1e266b80de32e07d5d6721

Before this pull request, goreleaser check printed this:

$goreleaser check
    • loading config file       file=.goreleaser.yml
    ⨯ command failed            error=yaml: unmarshal errors:
  line 67: field files not found in type config.NFPM

`nfpms.files` is deprecated: <https://goreleaser.com/deprecations/#nfpmsfiles>

```shell
goreleaser version 0.172.1
commit: 32a44ab928879bb32c1e266b80de32e07d5d6721
```

Before this commit, `goreleaser check` prints this:

```shell
$goreleaser check
   • loading config file       file=.goreleaser.yml
   ⨯ command failed            error=yaml: unmarshal errors:
  line 67: field files not found in type config.NFPM
```
@chemotaxis
Copy link
Contributor Author

Should I also update GoReleaser from v0.169.0 to v0.172.1?

@samcoe
Copy link
Contributor

samcoe commented Jul 1, 2021

@chemotaxis I think updating to 0.172.1 makes sense if you are up for including that in this PR.

@chemotaxis chemotaxis changed the title Update GoReleaser config file Update GoReleaser to v0.172.1 Jul 1, 2021
Now that the config file is updated, upgrade from v0.169.0 to v0.172.1.
Copy link
Contributor

@mislav mislav left a comment

Choose a reason for hiding this comment

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

Looks great to me! Thanks for addressing the breakage 🙇

@samcoe samcoe merged commit 6d0fb9b into cli:trunk Jul 15, 2021
@chemotaxis chemotaxis deleted the update-goreleaser-20210630 branch July 19, 2021 02:11
This was referenced Jul 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

upgrade goreleaser

3 participants