-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
Deterministic output for doc types #76890
Deterministic output for doc types #76890
Conversation
They generate non-deterministic types microsoft/rushstack#1958
Pinging @elastic/kibana-platform (Team:Platform) |
💚 Build SucceededBuild metrics
To update your PR or re-run it, just comment with: |
"incremental": false, | ||
"declaration": true, |
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.
Just to understand, if it's only impacting types / doc generation, why isn't this single change enough?
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.
incremental: false
here fixed the problem only for types inferred within the project. However, logging.appenders
unfolds to @kbn/config-schema
type https://github.com/elastic/kibana/pull/76890/files#diff-66a08ea686ee74b5fa19fbb89c8c7803L366 @kbn/config-schema/tsconfig.json
has incremental builds enabled. I decided that I'd rather specify an explicit type for this case than disable incremental builds for the @kbn/config-schema
package.
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.
Ok, thanks for the explanation
* disable incremental builds for public type generation. They generate non-deterministic types microsoft/rushstack#1958 * do not infer public types
* disable incremental builds for public type generation. They generate non-deterministic types microsoft/rushstack#1958 * do not infer public types
Summary
tsc
with incremental builds enabled generates non-deterministic types microsoft/rushstack#1958To make resulting API docs predictable, I did:
@elastic/kibana-app-arch FYI