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.

Sign up for GitHub

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

Jump to bottom

[core] Batch small changes #23327

Merged
merged 11 commits into from
Nov 1, 2020
Merged

[core] Batch small changes #23327

merged 11 commits into from
Nov 1, 2020

Conversation

oliviertassinari
Copy link
Member

@oliviertassinari oliviertassinari commented Oct 31, 2020 •
edited
Loading

  • [test] Skip visual regression not needed 31332ca: Some of the screenshots aren't necessary. Removing them helps with making the CI feedback loop faster. Also, we have x3 the number of workers for Argos-CI. Hopefully, it will no longer be the last CI report we wait for x3 more workers argos-ci/argos#430.
  • [docs] Add missing table badges: I have forgotten it in the initial pull request 0c28f8d
  • [docs] React vs object 6aee433
  • [docs] Link the source used in Wikipedia b5f3019
  • [TreeView] TypeScript convention 19bd302: types are shared in a global namespace.
  • [docs] Typo 4ea8e7e
  • [docs] Apply heading convention: convention defined in 2536c3c
  • [docs] Clear the different between table vs data grid right at the start f099bd0
  • [docs] Fix format of codesnippet 3a5cf49

@oliviertassinari oliviertassinari added umbrella For grouping multiple issues to provide a holistic view core Infrastructure work going on behind the scenes labels Oct 31, 2020
@mui-pr-bot
Copy link

mui-pr-bot commented Oct 31, 2020 •
edited
Loading

Details of bundle changes

Generated by 🚫 dangerJS against ff8460e

@github-actions github-actions bot mentioned this pull request Oct 31, 2020
Closed
eps1lon
eps1lon previously requested changes Oct 31, 2020
View reviewed changes
packages/material-ui/src/Paper/Paper.test.js Outdated Show resolved Hide resolved
@oliviertassinari oliviertassinari changed the title [core] Batch small changer [core] Batch small changes Oct 31, 2020
@oliviertassinari @mbrookes
Matt's review
ff8460e 
Co-authored-by: Matt <github@nospam.33m.co>
@oliviertassinari oliviertassinari merged commit 15fe791 into mui:next Nov 1, 2020
@oliviertassinari oliviertassinari deleted the batch-small-changer-v44 branch November 1, 2020 14:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mbrookes mbrookes mbrookes left review comments

@eps1lon eps1lon eps1lon approved these changes

Assignees
No one assigned
Labels
core Infrastructure work going on behind the scenes umbrella For grouping multiple issues to provide a holistic view
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@oliviertassinari @mui-pr-bot @mbrookes @eps1lon