Skip to content

fix: Stop if PreResourceResolver fails#251

Merged
kodiakhq[bot] merged 2 commits intocloudquery:mainfrom
disq:fix/stop-if-preresourceresolver-fails
Oct 6, 2022
Merged

fix: Stop if PreResourceResolver fails#251
kodiakhq[bot] merged 2 commits intocloudquery:mainfrom
disq:fix/stop-if-preresourceresolver-fails

Conversation

@disq
Copy link
Copy Markdown
Member

@disq disq commented Oct 5, 2022

If we continue the resolving after PreRR error, more (this time type based) errors are generated from the column resolvers that expect a certain type.

@disq disq requested a review from yevgenypats as a code owner October 5, 2022 11:46
@github-actions github-actions bot added the fix label Oct 5, 2022
@kodiakhq kodiakhq bot merged commit ee83f8f into cloudquery:main Oct 6, 2022
@disq disq deleted the fix/stop-if-preresourceresolver-fails branch October 6, 2022 09:47
kodiakhq bot pushed a commit that referenced this pull request Oct 7, 2022
🤖 I have created a release *beep* *boop*
---


## [0.12.8](v0.12.7...v0.12.8) (2022-10-07)


### Bug Fixes

* Error on incorrect table configuration ([#237](#237)) ([6ad75f5](6ad75f5))
* Exit gracefully on context cancelled ([#252](#252)) ([b4df92e](b4df92e))
* Progressbar should go into stdout ([#250](#250)) ([b8bcdad](b8bcdad))
* Recover panic in table resolver and object resolver flows ([#257](#257)) ([04dba02](04dba02))
* Stop if PreResourceResolver fails ([#251](#251)) ([ee83f8f](ee83f8f))

---
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 join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants