Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

reneleonhardt
Copy link
Contributor

Chores

  • dependabot auto-merge regenerates and commits dist/ if changed

@reneleonhardt
Copy link
Contributor Author

reneleonhardt commented Aug 14, 2025

This doesn't regenerate the already merged patches.
Alternatively I could merge that into an open pull request, for example typescript 5.9.

fetch-depth: 2
# if: steps.metadata.outputs.update-type == 'version-update:semver-patch'
- name: Check if package-lock.json has been changed
if: steps.metadata.outputs.update-type == 'version-update:semver-patch'
Copy link
Owner

Choose a reason for hiding this comment

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

I think you want to do this for all kinds of dependabot updates unconditionally

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I wouldn't do that, we already saw patches breaking CI 😞

And as I mentioned, currently there is no connection to any other CI jobs, even if any of them (eventually) fail this workflow will be executed, all run in parallel to each other.

Current conditions:

  • if a new PR has been created
  • if the author is dependabot
  • if update_type is patch

@reneleonhardt reneleonhardt force-pushed the chore/dependabot-regenerate branch from de9d902 to 97d397d Compare August 15, 2025 08:21
@Swatinem
Copy link
Owner

Ah yes, I understand now, this is only the automerge job, and minor/major bumps are not set up that way. so for those, it will still be necessary to manually regenerate those files, oh well :-(

@Swatinem Swatinem merged commit d8c5063 into Swatinem:master Aug 15, 2025
22 of 23 checks passed
@reneleonhardt
Copy link
Contributor Author

Well, there could be a new workflow which regenerates dist/ for all pull requests if package-lock.json has been changed 😄
The intention here was just to reduce the review load by auto-merging all patches...

@reneleonhardt reneleonhardt deleted the chore/dependabot-regenerate branch August 15, 2025 13:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants