Skip to content

fix: Bug where first resource wasn't insert into DB#258

Merged
shimonp21 merged 2 commits intocloudquery:mainfrom
shimonp21:fix_missing_first_resource
Oct 7, 2022
Merged

fix: Bug where first resource wasn't insert into DB#258
shimonp21 merged 2 commits intocloudquery:mainfrom
shimonp21:fix_missing_first_resource

Conversation

@shimonp21
Copy link
Copy Markdown
Contributor

Summary


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 🧪
  • Ensure the status checks below are successful ✅

@shimonp21 shimonp21 requested a review from yevgenypats as a code owner October 7, 2022 09:18
@github-actions github-actions bot added the fix label Oct 7, 2022
@shimonp21 shimonp21 enabled auto-merge (squash) October 7, 2022 09:19
@shimonp21 shimonp21 merged commit 2f5b78d into cloudquery:main Oct 7, 2022
shimonp21 pushed a commit that referenced this pull request Oct 7, 2022
🤖 I have created a release *beep* *boop*
---


##
[0.12.9](v0.12.8...v0.12.9)
(2022-10-07)


### Bug Fixes

* Bug where first resource wasn't insert into DB
([#258](#258))
([2f5b78d](2f5b78d))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
}()

var resource *schema.Resource
if err := json.Unmarshal(r.Resource, &resource); err != nil {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

double pointer here?

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