-
Notifications
You must be signed in to change notification settings - Fork 64.1k
Updated dependabot's docs to add Gradle Wrapper
#39954
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
base: main
Are you sure you want to change the base?
Conversation
|
Thanks for opening this pull request! A GitHub docs team member should be by to give feedback soon. In the meantime, please check out the contributing guidelines. |
How to review these changes 👓Thank you for your contribution. To review these changes, choose one of the following options: A Hubber will need to deploy your changes internally to review. Table of review linksNote: Please update the URL for your staging server or codespace. The table shows the files in the
Key: fpt: Free, Pro, Team; ghec: GitHub Enterprise Cloud; ghes: GitHub Enterprise Server 🤖 This comment is automatically generated. |
|
@gmazzo Thanks for opening a PR! 🎉 It looks like this is still a draft. Could you ping me when this is ready? Then, I'll get it up for review ⚡ |
|
A stale label has been added to this pull request because it has been open 30 days with no activity. If you think this pull request should remain open, please add a new comment. |
|
This is still waiting for dependabot/dependabot-core#12891 |
|
I think that we may want to leave this as a draft as long as this is an experiment as it may cause confusion otherwise |
I was planning to move it to ready once the main PR is merged, and just let them decide when to merge this one. But I don't know how coordinated are dependabot and GH docs people 🤔 |
It is a mutual effort, but I don't think that folks here will be tracking things like feature flags before deciding when to merge. As per the plan, the goal is to enable globally in ~1 week if things go as planned. I'd suggest to not merge this yet to avoid confusions for non-beta-enabled users You've done wonderful work so far. I can help track when to enable it here 🚀 |
Why:
dependabot/dependabot-core#12891 requires also to mention in the docs that now we'll support Gradle Wrapper in the
gradlemanagerCloses:
What's being changed (if available, include any code snippets, screenshots, or gifs):
Check off the following: