Skip to content
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.

Sign up for GitHub

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

Jump to bottom

feat: upgrade sdk, add triggering_actor to workflow_run table, etc #332

Merged
merged 4 commits into from
Sep 13, 2023
Merged

feat: upgrade sdk, add triggering_actor to workflow_run table, etc #332

merged 4 commits into from
Sep 13, 2023

Conversation

graza-io
Copy link
Contributor

No description provided.

@graza-io
Copy link
Contributor Author

This PR:

  • Upgrades go-github sdk to v55.0.0 (from v48.1.0)
  • Adds actor, actor_login, triggering_actor & triggering_actor_login to the github_actions_repository_workflow_run table.
  • BREAKING changes to both github_organization_dependabot_alert & github_repository_dependabot_alert tables
    • Removes security_advisory_cwes_cweid (string) & security_advisory_cwes_name (string) columns as new SDK version provides collection rather than first match only.
    • Adds security_advisory_cwes (JSON) to provide new collection from SDK.

@graza-io graza-io marked this pull request as ready for review September 11, 2023 10:30
@graza-io graza-io mentioned this pull request Sep 11, 2023
Merged
@cbruno10
Copy link
Contributor

@graza-io Do any of the new columns contain paginated information?

@cbruno10 cbruno10 mentioned this pull request Sep 12, 2023
Closed
@graza-io
Copy link
Contributor Author

@graza-io Do any of the new columns contain paginated information?

No.

@misraved misraved merged commit 41f0440 into main Sep 13, 2023
1 check passed
@misraved misraved deleted the issue-314 branch September 13, 2023 12:00
@misraved misraved mentioned this pull request Sep 13, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@misraved misraved misraved approved these changes

@cbruno10 cbruno10 Awaiting requested review from cbruno10

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@graza-io @cbruno10 @misraved