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

renovate: mobile ignore *_plus plugins #8415

Closed
wants to merge 1 commit into from

Conversation

shenlong-tanwen
Copy link
Member

Changes made

  • *_plus plugins are ignored from renovate updates until the jdk is migrated to 17

plus plugins are ignored from renovate updates until the jdk is migrated to 17
Copy link

cloudflare-pages bot commented Mar 31, 2024

Deploying immich with  Cloudflare Pages  Cloudflare Pages

Latest commit: f69dbde
Status: ✅  Deploy successful!
Preview URL: https://85231d32.immich.pages.dev
Branch Preview URL: https://renovate-ignore-plus-plugins.immich.pages.dev

View logs

@jrasm91
Copy link
Contributor

jrasm91 commented Mar 31, 2024

You can instead just close the pull requests.

@shenlong-tanwen
Copy link
Member Author

shenlong-tanwen commented Mar 31, 2024

You can instead just close the pull requests.

Yes but it'll take a while for us to update our JDK to Java 11. None of the other plugins have such weird build requirement. We can remove this ignore rule when we update them.

Edit: Oh, I just realized that closing PRs raised by renovate automatically puts it in an ignore list until we manually upgrade it to that version. Then maybe we can handle it as such and close this PR then.

@jrasm91
Copy link
Contributor

jrasm91 commented Mar 31, 2024

That's what I was thinking. It will still track the dependency and that way we don't forget about it, but we can ignore it until the sdk is updated.

@jrasm91 jrasm91 deleted the renovate/ignore-plus-plugins branch April 2, 2024 06:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants