Skip to content

chore: Update Go Releaser commands to not use deprecated features#14520

Merged
erezrokah merged 2 commits intocloudquery:mainfrom
erezrokah:chore/update_go_releaser_usage_deprecated
Oct 13, 2023
Merged

chore: Update Go Releaser commands to not use deprecated features#14520
erezrokah merged 2 commits intocloudquery:mainfrom
erezrokah:chore/update_go_releaser_usage_deprecated

Conversation

@erezrokah
Copy link
Copy Markdown
Member

Summary

We've been getting warnings from Go Releaser during publish since we use deprecated features:

  1. brews.tap was renamed to brews.repository: https://goreleaser.com/deprecations/#krewsindex
  2. --skip-* was merged into a single flag: https://goreleaser.com/deprecations/#active-deprecation-notices

@erezrokah erezrokah changed the title chore: Update Go Releaser command to not use deprecated features chore: Update Go Releaser commands to not use deprecated features Oct 13, 2023
@cq-bot cq-bot added the ci label Oct 13, 2023
@erezrokah
Copy link
Copy Markdown
Member Author

Tests are failing since I opened this PR from a fork so it's missing some secrets to run the tests

@erezrokah erezrokah merged commit 70e16f7 into cloudquery:main Oct 13, 2023
hydratim pushed a commit to hydratim/cloudquery that referenced this pull request Oct 20, 2023
…oudquery#14520)

<!-- 🎉 Thank you for making CloudQuery awesome by submitting a PR 🎉 -->

#### Summary

We've been getting warnings from Go Releaser during publish since we use
deprecated features:
1. `brews.tap` was renamed to `brews.repository`:
https://goreleaser.com/deprecations/#krewsindex
2. `--skip-*` was merged into a single flag:
https://goreleaser.com/deprecations/#active-deprecation-notices

<!--
Use the following steps to ensure your PR is ready to be reviewed

- [ ] Read the [contribution guidelines](../blob/main/CONTRIBUTING.md)
🧑‍🎓
- [ ] Run `make lint` to ensure the proposed changes follow the coding
style 🚨 (install golangci-lint
[here](https://golangci-lint.run/usage/install/#local-installation))
- [ ] Run `make test` to ensure the proposed changes pass the tests 🧪
- [ ] If changing a source plugin run `make gen` to ensure docs are up
to date 📝
- [ ] Ensure the status checks below are successful ✅
--->
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.

3 participants