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

fix: Add uv to fix disparity between Dependabot docs articles #37262

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

adavis444
Copy link

Why:

Closes: #37261

What's being changed (if available, include any code snippets, screenshots, or gifs):

Add this line to the package-ecosystem table:

| uv | `uv` | Not applicable |

Check off the following:

  • A subject matter expert (SME) has reviewed the technical accuracy of the content in this PR. In most cases, the author can be the SME. Open source contributions may require an SME review from GitHub staff.
  • The changes in this PR meet the docs fundamentals that are required for all content.
  • All CI checks are passing and the changes look good in the review environment.

@Copilot Copilot bot review requested due to automatic review settings April 2, 2025 14:37
Copy link

welcome bot commented Apr 2, 2025

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.

Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR addresses a disparity in the Dependabot documentation by adding the missing "uv" package ecosystem entry to the options reference file.

  • Updated the package-ecosystem table to include "uv".
  • Aligns the content with the corresponding entry in the reusables documentation file.

@github-actions github-actions bot added the triage Do not begin working on this issue until triaged by the team label Apr 2, 2025
Copy link
Contributor

github-actions bot commented Apr 2, 2025

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 links

⚠️ Warning: Our review server is experiencing latency issues.

The table shows the files in the content directory that were changed in this pull request. This helps you review your changes on the review server. Changes to the data directory are not included in this table.

Source Review Production What Changed
code-security/dependabot/working-with-dependabot/dependabot-options-reference.md fpt
ghec
ghes@ 3.16 3.15 3.14 3.13 3.12
fpt
ghec
ghes@ 3.16 3.15 3.14 3.13 3.12

Key: fpt: Free, Pro, Team; ghec: GitHub Enterprise Cloud; ghes: GitHub Enterprise Server

🤖 This comment is automatically generated.

@Sharra-writes Sharra-writes added content This issue or pull request belongs to the Docs Content team dependabot Content related to Dependabot and removed triage Do not begin working on this issue until triaged by the team labels Apr 2, 2025
@Sharra-writes
Copy link
Contributor

Sharra-writes commented Apr 4, 2025

@adavis444 Ah, I got so caught up in updating labels I forgot to thank you for opening this PR, and to let you know I'd get it put up for review! Sorry about that! I sent it to the team for a look when I updated the labels.

@Sharra-writes Sharra-writes added the needs SME This proposal needs review from a subject matter expert label Apr 4, 2025
Copy link
Contributor

github-actions bot commented Apr 4, 2025

Thanks for opening a pull request! We've triaged this issue for technical review by a subject matter expert 👀

@@ -399,6 +399,7 @@ Package manager | YAML value | Supported versions |
| pub | `pub` | v2 |
| Swift | `swift` | v5 |
| Terraform | `terraform` | >= 0.13, <= 1.10.x |
| uv | `uv` | Not applicable |
Copy link
Member

Choose a reason for hiding this comment

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

I don't think Not applicable is quite right, since uv may introduce breaking changes in their next major release and we obviously don't want to promise we'll have immediate support for those...

Given uv is currently on 0.6.12, does v0 make sense?

Suggested change
| uv | `uv` | Not applicable |
| uv | `uv` | v0 |

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content This issue or pull request belongs to the Docs Content team dependabot Content related to Dependabot needs SME This proposal needs review from a subject matter expert
Projects
None yet
Development

Successfully merging this pull request may close these issues.

uv present in supported-package-managers.md but absent in dependabot-options-reference.md
3 participants