Skip to content

Conversation

timotheeguerin
Copy link
Member

@timotheeguerin timotheeguerin commented Jul 24, 2025

fix #3020

typespec-azure with azure-specs repo check Azure/typespec-azure#3057

Copy link

pkg-pr-new bot commented Jul 24, 2025

Open in StackBlitz

npm i https://pkg.pr.new/microsoft/typespec/@typespec/versioning@7999

commit: 1bfa317

Copy link
Contributor

github-actions bot commented Jul 24, 2025

All changed packages have been documented.

  • @typespec/versioning
Show changes

@typespec/versioning - feature ✏️

Use of @useDependency is now optional when referencing types from a versioned library. By default the latest version of the library will be used.

@azure-sdk
Copy link
Collaborator

azure-sdk commented Jul 24, 2025

You can try these changes here

🛝 Playground 🌐 Website 🛝 VSCode Extension

@timotheeguerin timotheeguerin marked this pull request as ready for review July 24, 2025 20:53
Copy link
Contributor

@markcowl markcowl left a comment

Choose a reason for hiding this comment

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

This looks great. I wonder how much the cache will improve performance in some complex specs. I think we probably need to try this against the specs repo to ensure thare aren't any weird side-effects

@timotheeguerin
Copy link
Member Author

This looks great. I wonder how much the cache will improve performance in some complex specs. I think we probably need to try this against the specs repo to ensure thare aren't any weird side-effects

I doin't think the cache will do anything, that was something used just for the validation before so didn't need to be cached.

@timotheeguerin timotheeguerin added this pull request to the merge queue Jul 29, 2025
Merged via the queue into microsoft:main with commit 2fc988b Jul 29, 2025
27 checks passed
@timotheeguerin timotheeguerin deleted the optional-use-dependency branch July 29, 2025 18:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove @useDependency requirement for libraries
3 participants