-
Notifications
You must be signed in to change notification settings - Fork 594
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
[api-extractor] Fix an issue where .d.ts trimming did not work for exported variable declaration #945
Conversation
@@ -219,6 +219,8 @@ export declare class TypeReferencesInAedoc { | |||
|
|||
declare const unexportedCustomSymbol: unique symbol; | |||
|
|||
/* Excluded from this release type: VARIABLE */ |
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.
Is this right? Shouldn't there be a variable here? #Resolved
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.
It is marked as /** @alpha */
whereas you're looking at the dist/public output #Resolved
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.
Approved with suggestions.
@@ -246,6 +246,8 @@ export declare class TypeReferencesInAedoc { | |||
|
|||
declare const unexportedCustomSymbol: unique symbol; | |||
|
|||
export declare const VARIABLE: string; |
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.
Should this have a comment? #WontFix
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.
Good point.
I have fixed this issue in my AE7 branch, but I won't backport the fix to AE6, because in this branch the _getReleaseTagForDeclaration() logic doesn't save the comment that it found.
AE7 will be out soon.
In reply to: 234018505 [](ancestors = 234018505)
Fixes GitHub #936