Skip to content
This repository has been archived by the owner on Aug 16, 2022. It is now read-only.

fix(errors): Use WithNoOverwrite instead of WithOptionalSeverity #975

Merged
merged 3 commits into from
Jun 1, 2022

Conversation

erezrokah
Copy link
Member

@erezrokah erezrokah commented Jun 1, 2022

🎉 Thank you for making CloudQuery awesome by submitting a PR 🎉

Summary

Replaces #971.
Reverts #906 but keeps the fix for #869 using WithNoOverwrite.

See more in cloudquery/cloudquery#1384


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

  • Read the contribution guidelines 🧑‍🎓
  • Run go fmt to format your code 🖊
  • Lint your changes via golangci-lint run 🚨 (install golangci-lint here)
  • Update or add tests. Learn more about testing here 🧪
  • Update the docs by running go run ./docs/docs.go and committing the changes 📃
  • If adding a new resource, add relevant Terraform files in a separate PR 📂
  • Ensure the status checks below are successful ✅

@erezrokah
Copy link
Member Author

Opened cloudquery/cloudquery#1384 as a follow up

@erezrokah erezrokah merged commit e6d2086 into cloudquery:main Jun 1, 2022
@erezrokah erezrokah deleted the fix/use_no_overwrite branch June 1, 2022 06:51
kodiakhq bot pushed a commit that referenced this pull request Jun 1, 2022
🤖 I have created a release *beep* *boop*
---


### [0.12.2](v0.12.1...v0.12.2) (2022-06-01)


### Bug Fixes

* **deps:** Update module github.com/cloudquery/cq-provider-sdk to v0.10.6 ([#977](#977)) ([2baacf2](2baacf2))
* **errors:** Use `WithNoOverwrite` instead of `WithOptionalSeverity` ([#975](#975)) ([e6d2086](e6d2086))
* Fixed `NotFound` errors exceptions for some resources ([#965](#965)) ([d2cab56](d2cab56))
* Update endpoints ([#972](#972)) ([2af266a](2af266a))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants