fluentui icon indicating copy to clipboard operation
fluentui copied to clipboard

tools: format/order particular type of files automatically

Open Hotell opened this issue 4 years ago • 13 comments

nx is properly re-ordering/formatting tsconfig.json.base#paths, nx.json, workspace.json after any kind of generator is invoked that creates/updates/removes a package to maintain consistency and to reduce merge conflicts.

We should extend this behaviour and introduce similar automation to our particular needs.

Candidates (we will update this list as needed):

  • any package.json (scripts,dependencies) should be sorted to follow npm/yarn ordering when modified manually or by generator
  • ADD ANOTHER ONE

How should this work:

  • ideally during pre or push commit hook in the background
  • CI should fail on any kind of violations to not get out of sync if some power user will use --no-verify while running git commands

nit: This dependency should go on line 28 to maintain alphabetical order.

Originally posted by @khmakoto in https://github.com/microsoft/fluentui/pull/19517#discussion_r697619992

Hotell avatar Aug 30 '21 18:08 Hotell

Because this issue has not had activity for over 180 days, we're automatically closing it for house-keeping purposes.

Still require assistance? Please, create a new issue with up-to date details.

msft-fluent-ui-bot avatar Mar 01 '22 19:03 msft-fluent-ui-bot

Because this issue has not had activity for over 180 days, we're automatically closing it for house-keeping purposes.

Still require assistance? Please, create a new issue with up-to date details.

msft-fluent-ui-bot avatar Aug 29 '22 10:08 msft-fluent-ui-bot

Because this issue has not had activity for over 180 days, we're automatically closing it for house-keeping purposes.

Still require assistance? Please, create a new issue with up-to date details.

Because this issue has not had activity for over 180 days, we're automatically closing it for house-keeping purposes.

Still require assistance? Please, create a new issue with up-to date details.

Because this issue has not had activity for over 180 days, we're automatically closing it for house-keeping purposes.

Still require assistance? Please, create a new issue with up-to date details.

Because this issue has not had activity for over 180 days, we're automatically closing it for house-keeping purposes.

Still require assistance? Please, create a new issue with up-to date details.

Because this issue has not had activity for over 180 days, we're automatically closing it for house-keeping purposes.

Still require assistance? Please, create a new issue with up-to date details.

Because this issue has not had activity for over 180 days, we're automatically closing it for house-keeping purposes.

Still require assistance? Please, create a new issue with up-to date details.

Because this issue has not had activity for over 180 days, we're automatically closing it for house-keeping purposes.

Still require assistance? Please, create a new issue with up-to date details.

Because this issue has not had activity for over 180 days, we're automatically closing it for house-keeping purposes.

Still require assistance? Please, create a new issue with up-to date details.

Because this issue has not had activity for over 180 days, we're automatically closing it for house-keeping purposes.

Still require assistance? Please, create a new issue with up-to date details.

This issue has not had activity for over 180 days! We're adding Soft close label and will close it soon for house-keeping purposes. Still require assistance? Please add comment - "keep open".

keep open

Hotell avatar Oct 07 '24 10:10 Hotell

This issue has not had activity for over 180 days! We're adding Soft close label and will close it soon for house-keeping purposes. Still require assistance? Please add comment - "keep open".

keep open

Hotell avatar Apr 07 '25 09:04 Hotell

This issue has not had activity for over 180 days! We're adding Soft close label and will close it soon for house-keeping purposes. Still require assistance? Please add comment - "keep open".