Skip to content

Add support for record_ecosystem_meta output; Renable enable-record-ecosystem-meta experiment by default#1611

Merged
mburumaxwell merged 1 commit intomburumaxwell:mainfrom
rhyskoedijk:feature/enable-record-ecosystem-meta
May 14, 2025
Merged

Add support for record_ecosystem_meta output; Renable enable-record-ecosystem-meta experiment by default#1611
mburumaxwell merged 1 commit intomburumaxwell:mainfrom
rhyskoedijk:feature/enable-record-ecosystem-meta

Conversation

@rhyskoedijk
Copy link
Copy Markdown
Contributor

@rhyskoedijk rhyskoedijk commented Mar 11, 2025

This is the long-term fix for #1537, see #1537 (comment) for more context.

dependabot/cli#407 will add support for the record_ecosystem_meta output in Dependabot-CLI. After it is merged, the enable-record-ecosystem-meta experiment will be reenabled, which is what GitHub Dependabot does.

Ultimately, this change adds zero extra functionality to the DevOps Dependabot implementation. However, it does:

  1. Keep the behaviour between GitHub and DevOps aligned
  2. Remove the special handling that was previously required for this experiment
  3. Prevent experiments.ts: disable 'enable-record-ecosystem-meta' #1537 from regressing if the dependabot-core team decide to bake this experiment in to the core code through a future update

@rhyskoedijk rhyskoedijk force-pushed the feature/enable-record-ecosystem-meta branch from ee15895 to 4a839d8 Compare April 6, 2025 21:04
@rhyskoedijk
Copy link
Copy Markdown
Contributor Author

This can be merged now record_ecosystem_meta is supported as of CLI v1.63.0

@rhyskoedijk rhyskoedijk marked this pull request as ready for review May 7, 2025 22:35
@mburumaxwell mburumaxwell merged commit 76b242c into mburumaxwell:main May 14, 2025
8 checks passed
@rhyskoedijk rhyskoedijk deleted the feature/enable-record-ecosystem-meta branch May 14, 2025 21:31
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