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

[dev-tool] use package scope when generating sample apiref link #32465

Merged
merged 17 commits into from
Jan 10, 2025

Conversation

jeremymeng
Copy link
Member

Currently the apiref link template has a hardcoded @azure scope so it doesn't
work for packages with @azure-rest scope. Those package relies on the
customized apiRefLink fields in the samples configuration section of
package.json. This PR adds package scope variable to the apiref link template as
well so the link generatation works for both @azure and @azure-rest scopes.
These two scopes are the only ones that have SDK API reference pages currently.

With this change, customized apiRefLink for -rest packages are also removed.

Currently the apiref link template has a hardcoded `@azure` scope so it doesn't
work for packages with `@azure-rest` scope. Those package relies on the
customized `apiRef` fields in the samples configuration section of package.json.
This PR adds package scope variable to the apiref link template as well so the
link generatation works for both `@azure` and `@azure-rest` scopes.  These two
scopes are the only ones that have SDK API reference pages currently.
@github-actions github-actions bot added dev-tool Issues related to the Azure SDK for JS dev-tool Maps Mgmt This issue is related to a management-plane library. Purview labels Jan 8, 2025
@jeremymeng
Copy link
Member Author

FYI @kazrael2119

@azure-sdk
Copy link
Collaborator

API change check

API changes are not detected in this pull request.

@jeremymeng jeremymeng merged commit e8acf69 into Azure:main Jan 10, 2025
19 checks passed
@jeremymeng jeremymeng deleted the devtool/sample-api-ref-link branch January 10, 2025 00:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dev-tool Issues related to the Azure SDK for JS dev-tool Maps Mgmt This issue is related to a management-plane library. Purview
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants