Skip to content
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

[core] Batch small changes #24224

Merged
merged 5 commits into from Jan 3, 2021

Conversation

oliviertassinari
Copy link
Member

@oliviertassinari oliviertassinari commented Jan 2, 2021

  • Improve company address format fcea4e9: We used to mix a bunch of different combinations for the address. This is the version we agreed on with Matt a few weeks ago. We also agreed to fix issues lazily, as we spot them.
  • Polish the description of each package f903fdc: I have noticed that we were using the former description, that we never updated it:

Capture d’écran 2021-01-02 à 19 35 45

https://www.npmjs.com/search?q=material%20ui

  • Material-UI X issue template uses v5 which is not what we want 138161d: The new URL is used in [core] Batch small changes mui-x#800. It distinguish next from latest.
  • Handle tab-index as a number f973ebf: We had the same "small change" done in the past. I might have missed these occurrences or they were new ones. TypeScript considers them as number,

@oliviertassinari oliviertassinari added the umbrella For grouping multiple issues to provide a holistic view label Jan 2, 2021
@mui-pr-bot
Copy link

mui-pr-bot commented Jan 2, 2021

Details of bundle changes

Generated by 🚫 dangerJS against 0a4996a

@@ -3,13 +3,14 @@
"version": "5.0.0-alpha.21",
"private": false,
"author": "Material-UI Team",
"description": "Material-UI Unstyled - collection of unstyled React components.",
"description": "Accessible unstyled React components.",
Copy link
Member

Choose a reason for hiding this comment

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

Can we remove this? The term is used inflationary and lost its meaning a while ago. Instead of contributing to making it meaningless we can be confident enough to omit it.

Copy link
Member Author

Choose a reason for hiding this comment

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

"this": Are you referring to "Accessible"?

Suggested change
"description": "Accessible unstyled React components.",
"description": "Unstyled React components to build custom design systems on top of.",

@eps1lon
Copy link
Member

eps1lon commented Jan 2, 2021

Can we remove the emotion packages from the dependencies in the -latest template?

@oliviertassinari
Copy link
Member Author

Can we remove the emotion packages from the dependencies in the -latest template?

👍 removed

@oliviertassinari
Copy link
Member Author

oliviertassinari commented Jan 3, 2021

It seems that including the change of the package.json descriptions was too ambitious for a batch, I'm removing it. Best to move it to a standalone PR. I would expect little or no discussions required for a "batch".

@oliviertassinari oliviertassinari merged commit e56db23 into mui:next Jan 3, 2021
@oliviertassinari oliviertassinari deleted the batch-small-changes-40 branch January 3, 2021 17:09
oliviertassinari added a commit that referenced this pull request Jan 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
umbrella For grouping multiple issues to provide a holistic view
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants