Skip to content

Conversation

@Bibo-Joshi
Copy link
Member

This new workflow automatically copies versions changed by depandabot to the pre-commit config, the readme and any other file where the version might appear. It's a mit naive, b/c it really just searches for exact matches of the version number, but I guess the cases were we have that exact version number appear elsewhere in the repo are seldom enough. Plus, we can review.

Live examples of this workflow can be seen at https://github.com/Bibo-Joshi/actions-tests/pulls (where you can also see that I got this working almost immediately 🫠)

@Bibo-Joshi Bibo-Joshi added ⚙️ dependencies affected functionality: dependencies 🔗 github-actions related technology: github-actions labels May 20, 2023
@Bibo-Joshi Bibo-Joshi requested review from clot27 and harshil21 May 20, 2023 12:21
@github-actions github-actions bot removed the ⚙️ dependencies affected functionality: dependencies label May 20, 2023
Copy link
Member

@harshil21 harshil21 left a comment

Choose a reason for hiding this comment

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

wow nice! Can't we tell it which files to look for though, so error rate can become 0%

@Bibo-Joshi
Copy link
Member Author

That was also my inital thought, but that might lead to false negatives. As those can't be seen in a PR diff, I decided that false positives might be the better.

Copy link
Member

@harshil21 harshil21 left a comment

Choose a reason for hiding this comment

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

Alright then, LGTM

@Bibo-Joshi Bibo-Joshi merged commit 4c8d733 into master May 31, 2023
@Bibo-Joshi Bibo-Joshi deleted the post-process-depeandabot-prs branch May 31, 2023 20:19
@github-actions github-actions bot locked and limited conversation to collaborators Jun 8, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

🔗 github-actions related technology: github-actions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants