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

[Snyk + GH Actions] Update requirements #787

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

[Snyk + GH Actions] Update requirements #787

wants to merge 1 commit into from

Conversation

Jin-Sun-tts
Copy link
Contributor

Update requirements

  • Updated requirements.in + requirements.txt
  • Auto-generated by snyk.yml

@Jin-Sun-tts Jin-Sun-tts force-pushed the requirement-patches branch 3 times, most recently from f5546c4 to 032d6ad Compare December 28, 2024 12:05
@btylerburton
Copy link
Contributor

We will have to rewrite the gitbot to be owned by one of the team members who has verified commmits...

Screenshot 2025-01-08 at 17 54 44

@FuhuXia
Copy link
Member

FuhuXia commented Jan 14, 2025

we did an update before #792, it brought some breaking changes. We need to identify the breaking change before run another update.

@btylerburton
Copy link
Contributor

  1. Added SSH signing key to datagov-bot account. We shouldn't see anymore unverified commits in future.
  2. Squashed down the current commits to remove the unverified one.
  3. We need to triage breaking dep, as Fuhu has mentioned above, but at least this PR is clean now.

@FuhuXia FuhuXia marked this pull request as draft January 22, 2025 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@btylerburton btylerburton btylerburton approved these changes

Assignees
No one assigned
Labels
automated pr requirements snyk
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Jin-Sun-tts @btylerburton @FuhuXia @datagov-bot