Skip to content

fix(github): Fix panic in workflow fetching#6693

Merged
kodiakhq[bot] merged 3 commits intomainfrom
fix-github-panic
Jan 11, 2023
Merged

fix(github): Fix panic in workflow fetching#6693
kodiakhq[bot] merged 3 commits intomainfrom
fix-github-panic

Conversation

@hermanschaaf
Copy link
Copy Markdown
Contributor

  • Fixes bug(github): Nil pointer dereference #6667
  • Adds a repository column to the workflows table, which I figure is actually quite important to make it useful. I opted not to make it part of the PK, because 1) it doesn't seem necessary and 2) that would make it a breaking change

@hermanschaaf hermanschaaf requested review from a team and candiduslynx and removed request for a team January 11, 2023 13:51
@cq-bot cq-bot added the github label Jan 11, 2023
@github-actions
Copy link
Copy Markdown

This PR has the following changes to source plugin(s) tables:

  • Table github_workflows: column added with name repository and type String
  • Table github_workflows: column order changed for badge_url

@hermanschaaf hermanschaaf added the automerge Automatically merge once required checks pass label Jan 11, 2023
@kodiakhq kodiakhq bot merged commit a582f0b into main Jan 11, 2023
@kodiakhq kodiakhq bot deleted the fix-github-panic branch January 11, 2023 16:36
kodiakhq bot pushed a commit that referenced this pull request Jan 12, 2023
🤖 I have created a release *beep* *boop*
---


## [2.1.1](plugins-source-github-v2.1.0...plugins-source-github-v2.1.1) (2023-01-12)


### Bug Fixes

* **deps:** Update module github.com/cloudquery/plugin-sdk to v1.24.2 ([#6695](#6695)) ([694ab9f](694ab9f))
* **deps:** Update module github.com/cloudquery/plugin-sdk to v1.25.0 ([#6745](#6745)) ([9c41854](9c41854))
* **github:** Fix panic in workflow fetching ([#6693](#6693)) ([a582f0b](a582f0b))

---
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

Labels

automerge Automatically merge once required checks pass

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug(github): Nil pointer dereference

4 participants