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

Limit the timing of git submodule updates by renovate to avoid disrupting work during working hours #743

Merged
merged 1 commit into from
Jan 16, 2025

Conversation

Yang-33
Copy link
Contributor

@Yang-33 Yang-33 commented Jan 16, 2025

After updating line-openapi, there are times when both

  • (1) a PR generated after Renovate updates the git submodule and
  • (2) a PR created by an operator dispatching the GitHub workflow

are generated simultaneously.
This change is to avoid that. Usually, when it's known that the bot SDK code will be changed (which is most cases), (1) is not necessary because the operator dispatches the workflow to change the code.

In particular, since the operator usually dispatches the workflow for 6 bot sdk repositories via a script, the PRs automatically created by Renovate are a hindrance.

However, to account for the possibility that the operator might simply forget to update bot sdk code, I've restricted Renovate to update the git submodule only at night instead of not updating. This way, if code generation is forgotten, it can be noticed the next day.

(Other options) It was also possible to change the GitHub workflow to prevent duplicate PR by using same branch, but if the operator forgets to update for a while, there will be multiple features changed in one PR. I don't like this. though both options are fine, I think this PR is more suitable for our work at the moment rather than this option.

"extends": [
"config:base"
],
"timezone": "Asia/Tokyo",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

diff may not work but changes are just

  1. rename renovate.json to renovate.json5 to write comment
  2. add timezone and packageRules[].schedule

Yang-33 added a commit to line/line-bot-sdk-nodejs that referenced this pull request Jan 16, 2025
@Yang-33 Yang-33 merged commit cca1e70 into master Jan 16, 2025
18 checks passed
@Yang-33 Yang-33 deleted the update-git-submodule-at-night-by-ranovate branch January 16, 2025 02:47
Yang-33 added a commit to line/line-bot-sdk-php that referenced this pull request Jan 16, 2025
Yang-33 added a commit to line/line-bot-sdk-java that referenced this pull request Jan 16, 2025
Yang-33 added a commit to line/line-bot-sdk-go that referenced this pull request Jan 16, 2025
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