Skip to content

react-radio: write readme#22676

Merged
spmonahan merged 1 commit into
microsoft:masterfrom
spmonahan:radio/readme
Apr 27, 2022
Merged

react-radio: write readme#22676
spmonahan merged 1 commit into
microsoft:masterfrom
spmonahan:radio/readme

Conversation

@spmonahan

Copy link
Copy Markdown
Contributor

Current Behavior

react-radio only had the generated boilerplate readme.

New Behavior

react-radio now has a readme with basic usage and links to other relevant information.

Related Issue(s)

#19953

@codesandbox-ci

Copy link
Copy Markdown

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.

Latest deployment of this branch, based on commit dc8e40f:

Sandbox Source
@fluentui/react 8 starter Configuration
@fluentui/react-components 9 starter Configuration

@fabricteam

fabricteam commented Apr 27, 2022

Copy link
Copy Markdown
Collaborator

📊 Bundle size report

🤖 This report was generated against f0764e35df36354a86bcf44912a9bf8756df7ddd

@fabricteam

fabricteam commented Apr 27, 2022

Copy link
Copy Markdown
Collaborator

Perf Analysis (@fluentui/react-components)

No significant results to display.

All results

Scenario Render type Master Ticks PR Ticks Iterations Status
Avatar mount 794 840 5000
Button mount 540 555 5000
FluentProvider mount 1862 1766 5000
FluentProviderWithTheme mount 234 253 10
FluentProviderWithTheme virtual-rerender 232 207 10
FluentProviderWithTheme virtual-rerender-with-unmount 304 307 10
MakeStyles mount 1507 1582 50000

@size-auditor

size-auditor Bot commented Apr 27, 2022

Copy link
Copy Markdown

Asset size changes

Size Auditor did not detect a change in bundle size for any component!

Baseline commit: f0764e35df36354a86bcf44912a9bf8756df7ddd (build)

@spmonahan spmonahan mentioned this pull request Apr 27, 2022
6 tasks
@spmonahan

Copy link
Copy Markdown
Contributor Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 3 pipeline(s).

@@ -0,0 +1,7 @@
{
"type": "prerelease",
"comment": "write readme",

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit:

Suggested change
"comment": "write readme",
"comment": "Adding readme.",

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I went back n forth on this but technically it was already added and just written :)

@spmonahan spmonahan merged commit 5f27c51 into microsoft:master Apr 27, 2022
marwan38 pushed a commit to marwan38/fluentui that referenced this pull request Jun 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants