Skip to content

chore: update default versions to match generated code#8581

Merged
jskeet merged 1 commit into
googleapis:mainfrom
jskeet:fix-default-versions
Jun 11, 2026
Merged

chore: update default versions to match generated code#8581
jskeet merged 1 commit into
googleapis:mainfrom
jskeet:fix-default-versions

Conversation

@jskeet

@jskeet jskeet commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

This addresses situations where .repo-metadata.json and index.ts effectively disagree on the default API version for a package. The fix favors the version in index.ts, for the sake of compatibility.

Once this is merged, #8556, #8571 and #8480 should be regenerated.

Fixes googleapis/librarian#6382

@jskeet
jskeet requested review from pearigee and quirogas June 11, 2026 11:38
@jskeet
jskeet requested a review from a team as a code owner June 11, 2026 11:38

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request updates the default Node.js version from v1beta to v1 for several libraries in librarian.yaml. There are no review comments, and I have no feedback to provide.

Comment thread librarian.yaml Outdated
skip_generate: true
nodejs:
default_version: v1beta
default_version: v1

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This is wrong and I'm confused... looking again.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

(Basically workstations was already okay, but generated before we specified default versions. Will revert this bit.)

This addresses situations where .repo-metadata.json and index.ts
effectively disagree on the default API version for a package. The fix
favors the version in index.ts, for the sake of compatibility.

Once this is merged, googleapis#8556, googleapis#8571 and googleapis#8480 should be regenerated.

Fixes googleapis/librarian#6382
@jskeet
jskeet force-pushed the fix-default-versions branch from bbe0b20 to 30beb8b Compare June 11, 2026 11:48
@jskeet
jskeet merged commit ca2f391 into googleapis:main Jun 11, 2026
32 checks passed
@jskeet
jskeet deleted the fix-default-versions branch June 11, 2026 12:58
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.

nodejs: address inconsistency of default versions

2 participants