Skip to content

fix(tools): update CODEOWNERS relative path of moved package#22624

Merged
TristanWatanabe merged 6 commits into
microsoft:masterfrom
TristanWatanabe:move-generator-update-codeowners
Apr 26, 2022
Merged

fix(tools): update CODEOWNERS relative path of moved package#22624
TristanWatanabe merged 6 commits into
microsoft:masterfrom
TristanWatanabe:move-generator-update-codeowners

Conversation

@TristanWatanabe

Copy link
Copy Markdown
Member

Current Behavior:

The move-packages nx generator was not updating the CODEOWNERS file with the moved package's new relative path

New Behavior:

move-packages generator now updates CODEOWNERS file after package move.

Fixes:

Follow-up to #22506 (comment)

@fabricteam

fabricteam commented Apr 25, 2022

Copy link
Copy Markdown
Collaborator

📊 Bundle size report

🤖 This report was generated against fbe1a3be52a1dfce28f5eab1d1c3969bce6dce16

@codesandbox-ci

codesandbox-ci Bot commented Apr 25, 2022

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 e732115:

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

@size-auditor

size-auditor Bot commented Apr 25, 2022

Copy link
Copy Markdown

Asset size changes

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

Baseline commit: fbe1a3be52a1dfce28f5eab1d1c3969bce6dce16 (build)

@TristanWatanabe TristanWatanabe marked this pull request as ready for review April 25, 2022 16:06
@TristanWatanabe TristanWatanabe requested a review from a team as a code owner April 25, 2022 16:06
@TristanWatanabe TristanWatanabe changed the title move-packages generator: automatically update CODEOWNERS file path of moved package fix(tools) automatically update CODEOWNERS file path of moved package Apr 25, 2022
@TristanWatanabe TristanWatanabe changed the title fix(tools) automatically update CODEOWNERS file path of moved package fix(tools) : automatically update CODEOWNERS file path of moved package Apr 25, 2022
@TristanWatanabe TristanWatanabe changed the title fix(tools) : automatically update CODEOWNERS file path of moved package fix(tools) : update CODEOWNERS relative path of moved package Apr 25, 2022
@TristanWatanabe TristanWatanabe changed the title fix(tools) : update CODEOWNERS relative path of moved package fix(tools): update CODEOWNERS relative path of moved package Apr 25, 2022
Comment thread tools/generators/move-packages/index.spec.ts Outdated
Comment thread tools/generators/move-packages/index.spec.ts Outdated

@Hotell Hotell left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🙌

@TristanWatanabe TristanWatanabe merged commit b0ff453 into microsoft:master Apr 26, 2022
@TristanWatanabe TristanWatanabe deleted the move-generator-update-codeowners branch April 26, 2022 14:59
marwan38 pushed a commit to marwan38/fluentui that referenced this pull request Jun 13, 2022
…ft#22624)

* add updateCodeOwners function

* add test case

* fix test

* generalize update CODEOWNERS function

* nit: update var name

* update tests
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.

5 participants