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] #3465 incorrectly changed defaults for reportFolder
in the template
#4543
Comments
@octogonz - Which should be correct here? |
Hi all, just chiming in here because I stumbled on this in a new project I created. @mcdurdin is correct that the change made in #3465 is wrong and the PR should be rolled back. The default value as specified in the defaults is I suspect the confusion is coming from the fact that running api-extractor without the |
Summary
The default for reportFolder is
etc
:rushstack/apps/api-extractor/src/schemas/api-extractor-defaults.json
Line 20 in 8a33edf
The change in #3465 changed this to
temp
. This is wrong, and it makes the template comments confusing, because thetemp
folder should not be committed to git. The report file is written to both temp and etc on first run.Details
#3465 should be rolled back.
Standard questions
Please answer these questions to help us investigate your issue more quickly:
@microsoft/api-extractor
version?node -v
)?The text was updated successfully, but these errors were encountered: