Skip to content

fix(scripts-beachball): ignore web-components packages from v8 release scope#31667

Merged
Hotell merged 2 commits intomicrosoft:masterfrom
Hotell:releases/wrong-release-trigger
Jun 11, 2024
Merged

fix(scripts-beachball): ignore web-components packages from v8 release scope#31667
Hotell merged 2 commits intomicrosoft:masterfrom
Hotell:releases/wrong-release-trigger

Conversation

@Hotell
Copy link
Contributor

@Hotell Hotell commented Jun 11, 2024

Previous Behavior

web-components were not ignored within v8 release scope, thus invalid releases have been triggered every-time web-components added a change file.

Even more confusion was added to changelog.md v8 files which refers to web-component changes with date of web-component release.

Example:

https://github.com/microsoft/fluentui/blob/master/packages/react/CHANGELOG.md#81176 points to https://github.com/microsoft/fluentui/pull/31585/files which was released https://github.com/microsoft/fluentui/blob/master/packages/web-components/CHANGELOG.md#300-beta24

New Behavior

  • web-components tagged packages are excluded from v8 scope
  • web-components tagged packages are explicitly provided to web-components release-scope without ignoring all v9 tagged packages ( which is not needed )

Extra:

  • invalid changefiles are removed
image

Related Issue(s)

@github-actions github-actions bot added this to the April Project Cycle Q1 2024 milestone Jun 11, 2024
@Hotell Hotell changed the title Releases/wrong release trigger fix(scripts-beachball): ignore web-components packages from v8 release scope Jun 11, 2024
@fabricteam
Copy link
Collaborator

📊 Bundle size report

✅ No changes found

@Hotell Hotell marked this pull request as ready for review June 11, 2024 15:59
@Hotell Hotell requested a review from a team as a code owner June 11, 2024 15:59
@Hotell Hotell merged commit aba1bb6 into microsoft:master Jun 11, 2024
@Hotell Hotell deleted the releases/wrong-release-trigger branch June 11, 2024 17:09
marcosmoura added a commit to marcosmoura/fluentui that referenced this pull request Jun 12, 2024
…escription

* master:
  chore(react-list-preview): Add bundle size tests (microsoft#31658)
  applying package updates
  feat:react-nav-preview Add a handful of accessibility features for Nav (microsoft#31649)
  feat: refactor anchor-button to use element internals and anchor proxy element (microsoft#31653)
  chore:(docs) New Component doc (microsoft#31611)
  style(react-nav-preview): fix formatting in barel file (microsoft#31671)
  feat(web-components): refactor progress bar to use ElementInternals (microsoft#31652)
  chore(react-components): split react libraries in two (/library and /stories) - last batch (microsoft#31581)
  fix(scripts-beachball): ignore web-components packages from v8 release scope (microsoft#31667)
  chore(react-components): split react libraries in two (/library and /stories) - cxe-red /2nd batch (microsoft#31580)
  chore: Remove recipes-react-components in favor of recipes package (microsoft#31647)
  feat: Add consistent start and finish lifecycle callbacks (microsoft#31644)
  chore(docs): Remove requirement of performance tests from epic template (microsoft#31662)
  chore(react-components): split react libraries in two (/library and /stories) - cxe-red /1st batch (microsoft#31579)
  docs(react-swatch-picker): added SwatchPicker page to Component Mapping (microsoft#31637)
  docs(dev-env): update minimal node version (microsoft#31638)
marcosmoura added a commit to marcosmoura/fluentui that referenced this pull request Jun 12, 2024
* master:
  chore(react-list-preview): Add bundle size tests (microsoft#31658)
  applying package updates
  feat:react-nav-preview Add a handful of accessibility features for Nav (microsoft#31649)
  feat: refactor anchor-button to use element internals and anchor proxy element (microsoft#31653)
  chore:(docs) New Component doc (microsoft#31611)
  style(react-nav-preview): fix formatting in barel file (microsoft#31671)
  feat(web-components): refactor progress bar to use ElementInternals (microsoft#31652)
  chore(react-components): split react libraries in two (/library and /stories) - last batch (microsoft#31581)
  fix(scripts-beachball): ignore web-components packages from v8 release scope (microsoft#31667)
  chore(react-components): split react libraries in two (/library and /stories) - cxe-red /2nd batch (microsoft#31580)
  chore: Remove recipes-react-components in favor of recipes package (microsoft#31647)
  feat: Add consistent start and finish lifecycle callbacks (microsoft#31644)
  chore(docs): Remove requirement of performance tests from epic template (microsoft#31662)
  chore(react-components): split react libraries in two (/library and /stories) - cxe-red /1st batch (microsoft#31579)
  docs(react-swatch-picker): added SwatchPicker page to Component Mapping (microsoft#31637)
  docs(dev-env): update minimal node version (microsoft#31638)
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.

[Bug]: changelog links to incorrect PRs _or_ PRs are incorrectly bumping packages

4 participants