-
Notifications
You must be signed in to change notification settings - Fork 532
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
build: Update API-Extractor, remove now-unneeded pnpm patches, and update API reports to reflect bug fix #19939
Conversation
…date API reports to reflect bug fix
@@ -25,7 +25,6 @@ export class AttributableMap extends SharedObject<ISharedMapEvents> implements I | |||
[Symbol.iterator](): IterableIterator<[string, any]>; | |||
readonly [Symbol.toStringTag]: string; | |||
constructor(id: string, runtime: IFluidDataStoreRuntime, attributes: IChannelAttributes); | |||
// (undocumented) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Anything that was previously documented using {@inheritDoc }
pointing to a target in another library would be incorrectly annotated as (undocumented)
. This is now fixed!
@@ -122,7 +122,6 @@ | |||
"@fluid-private/readme-command": "workspace:~", | |||
"@fluidframework/build-common": "^2.0.3", | |||
"@fluidframework/eslint-config-fluid": "^4.0.0", | |||
"@microsoft/api-extractor": "^7.39.1", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was duplicated between dependencies
and devDependencies
. This package uses api-extractor in code, so the devDependencies
entry was removed.
⯅ @fluid-example/bundle-size-tests: +44 Bytes
Baseline commit: 7c4a08b |
…e tag compatibility (#20696) Mitigation for API-Extractor [issue 4430](microsoft/rushstack#4430). The original mitigation we had in place was erroneously removed in #19939. Also fixes the handful of violations that have been introduced since that PR.
…e tag compatibility (#20696) (#20699) Mitigation for API-Extractor [issue 4430](microsoft/rushstack#4430). The original mitigation we had in place was erroneously removed in #19939. Also fixes the handful of violations that have been introduced since that PR. Cherry-picked from #20696
Noteworthy changes beyond dependency version bumps:
bundledPackages
entries in API-Extractor configs to use glob patterns instead of regular expressions.{@inheritDoc}
comments as "undocumented" in API reports rushstack#3462(undocumented)
when they leveraged TSDoc{@inheritDoc }
comments in some cases.{@inheritDoc}
comments as "undocumented" in API reports rushstack#3462See API-Extractor's changelog here: https://github.com/microsoft/rushstack/blob/main/apps/api-extractor/CHANGELOG.md#7423