-
Notifications
You must be signed in to change notification settings - Fork 500
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat!: Bump go-github
to v59 in Github plugin
#16797
feat!: Bump go-github
to v59 in Github plugin
#16797
Conversation
This PR has the following changes to source plugin(s) tables:
|
go-github
to v59 in Github plugingo-github
to v59 in Github plugin
880f9d0
to
7455e8b
Compare
@@ -233,7 +233,7 @@ func githubClientForHTTPClient(httpClient *http.Client, logger zerolog.Logger, e | |||
} | |||
|
|||
if enterpriseSettings != nil { | |||
return github.NewEnterpriseClient(enterpriseSettings.BaseURL, enterpriseSettings.UploadURL, rateLimiter) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
NewEnterpriseClient
has been deprecated and throws errors when running make lint
, the replacement is to use WithEnterpriseURLs
go-github
to v59 in Github plugingo-github
to v59 in Github plugin
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks great @AshCorr 💯 Thanks for the PR.
The timestamp changes are not breaking for us as we already know to resolve GitHub timestamp representation as an arrow timestamp
timestamp := github.Timestamp{} |
the changes to float are indeed schema breaking (as we don't migrate changes in data types at the moment), so I've updated the PR title from feat:
to feat!:
to signal our automation to release this under a major version bump
…16796) #### Summary Add a `github_repository_sboms` table to sync data from [Github's Software Bill Of Materials (SBOM) API](https://docs.github.com/en/rest/dependency-graph/sboms?apiVersion=2022-11-28) This API can tell us which dependencies(NPM, Maven, etc) a repository has. With the Snyk plugin currently out of commission this provides a crucial way of gathering insights of which dependencies are being used across a Github organization. This PR includes commits from #16797 . I've seperated it into 2 PRs to reduce the cognitive load on reviewers, you may want to review this PR starting from [7d92c1d](7d92c1d) <img width="2034" alt="image" src="https://github.com/cloudquery/cloudquery/assets/21217225/c8da58e7-fb88-427e-8839-c3c920aa6a60"> <!-- Explain what problem this PR addresses --> <!--
🤖 I have created a release *beep* *boop* --- ## [8.0.0](plugins-source-github-v7.7.1...plugins-source-github-v8.0.0) (2024-02-21) ### ⚠ BREAKING CHANGES * Bump `go-github` to v59 in Github plugin ([#16797](#16797)) ### This Release has the Following Changes to Tables - Table `github_billing_action`: column type changed from `int64` to `float64` for `included_minutes` (:warning: breaking) - Table `github_billing_action`: column type changed from `int64` to `float64` for `total_minutes_used` (:warning: breaking) - Table `github_billing_package`: column type changed from `int64` to `float64` for `included_gigabytes_bandwidth` (:warning: breaking) - Table `github_billing_storage`: column type changed from `int64` to `float64` for `estimated_storage_for_month` (:warning: breaking) - Table `github_issues`: column added with name `draft` and type `bool` - Table `github_organization_dependabot_alerts`: column added with name `auto_dismissed_at` and type `timestamp[us, tz=UTC]` - Table `github_organization_dependabot_alerts`: column added with name `repository` and type `json` - Table `github_organizations`: column added with name `secret_scanning_validity_checks_enabled` and type `bool` - Table `github_repositories`: column added with name `custom_properties` and type `json` - Table `github_repositories`: column added with name `web_commit_signoff_required` and type `bool` - Table `github_repository_dependabot_alerts`: column added with name `auto_dismissed_at` and type `timestamp[us, tz=UTC]` - Table `github_repository_dependabot_alerts`: column added with name `repository` and type `json` - Table `github_repository_keys`: column added with name `added_by` and type `utf8` - Table `github_repository_keys`: column added with name `last_used` and type `timestamp[us, tz=UTC]` - Table `github_repository_sboms` was added - Table `github_team_repositories`: column added with name `custom_properties` and type `json` - Table `github_team_repositories`: column added with name `web_commit_signoff_required` and type `bool` - Table `github_workflow_jobs`: column added with name `created_at` and type `timestamp[us, tz=UTC]` - Table `github_workflow_jobs`: column added with name `head_branch` and type `utf8` - Table `github_workflow_jobs`: column added with name `workflow_name` and type `utf8` - Table `github_workflow_runs`: column added with name `display_title` and type `utf8` - Table `github_workflow_runs`: column added with name `referenced_workflows` and type `json` - Table `github_workflow_runs`: column added with name `triggering_actor` and type `json` ### Features * Add `github_repository_sboms` table to sync Github SBOM data ([#16796](#16796)) ([992434f](992434f)) * Bump `go-github` to v59 in Github plugin ([#16797](#16797)) ([5b89ef6](5b89ef6)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Summary
Bump
go-github
to v59 in the Github source plugin.This allows Cloudquery to take advantage of some of the new Github APIs, specifically the SBOM API in #16796
I've tested this PR in our Cloudquery environment and did not notice any issues arising from this bump, although to be fair we don't collect every table due to resource and time constraints.
Looking at the Release notes for go-github I don't see any breaking changes particularly concerning apart from a change of timestamp types in v50