-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
chore: apply single version policy on web-components domain #33660
base: master
Are you sure you want to change the base?
Conversation
@@ -112,7 +108,8 @@ | |||
"beachball": { | |||
"disallowedChangeTypes": [ | |||
"major", | |||
"minor" | |||
"minor", | |||
"patch" |
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.
ran into this during #33647
creating a patch change file would be breaking change as it would bump to non pre-release version
🕵 fluentui-web-components-v3 No visual regressions between this PR and main |
@@ -0,0 +1,7 @@ | |||
{ |
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.
🕵🏾♀️ visual regressions to review in the fluentuiv8 Visual Regression Report
Callout 1 screenshots
Image Name | Diff(in Pixels) | Image Type |
---|---|---|
Callout.No callout width specified.chromium.png | 2319 | Changed |
react-charting-LineChart 1 screenshots
Image Name | Diff(in Pixels) | Image Type |
---|---|---|
react-charting-LineChart.Gaps.chromium.png | 1 | Changed |
📊 Bundle size reportUnchanged fixtures
|
🕵 FluentUIV0 No visual regressions between this PR and main |
change/@fluentui-chart-web-components-5f931ecf-3d29-492f-814a-a717ff2375d9.json
Show resolved
Hide resolved
Pull request demo site: URL |
Previous Behavior
New Behavior
patch
change type onchart-web-components
Related Issue(s)