Skip to content

Conversation

@radium-v
Copy link
Contributor

Previous Behavior

The api-extractor step outputs a long list of warnings due to problems with documentation blocks in the component modules.

Additionally, Storybook outputs a long list of warnings for all exported types as values (import/export vs import type/export type).

New Behavior

This PR resolves the warnings and modifies the api-extractor and tsdoc configurations to reduce the number of warnings. Any warnings generated during the compile/build steps should be more relevant and actionable going forward.

Related Issue(s)

@fabricteam
Copy link
Collaborator

fabricteam commented May 23, 2024

📊 Bundle size report

✅ No changes found

@codesandbox-ci
Copy link

codesandbox-ci bot commented May 23, 2024

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

@radium-v radium-v force-pushed the users/radium-v/fix-warnings branch from 6359e58 to 3061f7a Compare May 23, 2024 22:45
@radium-v radium-v force-pushed the users/radium-v/fix-warnings branch from 3061f7a to 5bc2a4f Compare May 30, 2024 16:54
@radium-v radium-v marked this pull request as ready for review May 30, 2024 16:54
@radium-v radium-v requested review from a team as code owners May 30, 2024 16:54
@chrisdholt chrisdholt enabled auto-merge (squash) May 30, 2024 17:00
@Hotell
Copy link
Contributor

Hotell commented May 31, 2024

Any warnings generated during the compile/build steps should be more relevant and actionable going forward.

I'd suggest to set those to "logLevel": "error" so pipeline would fail, otherwise you'll create technical depth rather quickly from my experience

@radium-v radium-v force-pushed the users/radium-v/fix-warnings branch from 5bc2a4f to 8864258 Compare May 31, 2024 22:45
@chrisdholt chrisdholt merged commit 9b68140 into microsoft:master May 31, 2024
miroslavstastny pushed a commit to miroslavstastny/fluentui that referenced this pull request Jun 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants