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.

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

ci: automagically repackage when Dependabot updates a dependency #23

Merged
merged 1 commit into from
Feb 12, 2024

Conversation

dscho
Copy link
Member

@dscho dscho commented Feb 11, 2024

Dependabot is really cool and helpful, but sometimes it updates a dependency that requires npm run package to be run, and Dependabot does not have any way to know that that should happen.

Let's have an automated workflow do that, then.

This is copied from
https://github.com/git-for-windows/setup-git-for-windows-sdk/blob/190c5563/.github/workflows/npm-run-package.yml

@dscho dscho requested a review from rimrul February 11, 2024 15:16
@dscho dscho self-assigned this Feb 11, 2024
.github/workflows/npm-run-package.yml Outdated Show resolved Hide resolved
.github/workflows/npm-run-package.yml Outdated Show resolved Hide resolved
- run: npm ci
- run: npm run build
- run: npm run lint
- run: npm run format && git diff-files
Copy link
Member

@rimrul rimrul Feb 11, 2024

Choose a reason for hiding this comment

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

There is no format script in package.json

Copy link
Member Author

Choose a reason for hiding this comment

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

Done. There's also no npm run package.

Copy link
Member

Choose a reason for hiding this comment

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

The commot message still talks about npm run package

Copy link
Member Author

Choose a reason for hiding this comment

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

The commot message still talks about npm run package

Oops.

@dscho dscho force-pushed the npm-run-package-on-dependabot-prs branch from 2d1f1d2 to 0dad88c Compare February 11, 2024 21:55
Dependabot is really cool and helpful, but sometimes it updates a
dependency that requires `npm run prepare` to be run, and Dependabot
does not have any way to know that that should happen.

Let's have an automated workflow do that, then.

This is copied from
https://github.com/git-for-windows/setup-git-for-windows-sdk/blob/190c5563/.github/workflows/npm-run-package.yml

Signed-off-by: Johannes Schindelin <[email protected]>
@dscho dscho force-pushed the npm-run-package-on-dependabot-prs branch from 0dad88c to cc141fa Compare February 12, 2024 06:57
@dscho dscho merged commit 396a172 into git-for-windows:main Feb 12, 2024
1 check passed
@dscho dscho deleted the npm-run-package-on-dependabot-prs branch February 12, 2024 06:58
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