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

[test] Run the karma tests in browserstack #316

Merged
merged 4 commits into from
Sep 28, 2020

Conversation

oliviertassinari
Copy link
Member

@oliviertassinari oliviertassinari commented Sep 20, 2020

Cross-browser compatibility. We were missing the BrowserStack credentials in CircleCI.

Capture d’écran 2020-09-26 à 12 27 59

@oliviertassinari oliviertassinari changed the title run ci [test] Run the karma tests in browserstack Sep 20, 2020
@oliviertassinari
Copy link
Member Author

@dtassone The tests are run in:

Capture d’écran 2020-09-20 à 19 58 23

However, we have a couple of issues with these targets (e.g. Object.entries not defined). The grid isn't compatible. I think that it's important we configure browser to run the test in minimum browser versions we want to support. So, this time, we can't escape it: we decided to not support IE here (at least try and see the feedback), but what about the rest?

What's the minimum browsers' version we aim to support? :). It resonates with mui/material-ui#15496 from @eps1lon.

# # Default build command.
# command = "yarn docs:build && yarn docs:export && yarn storybook:build && yarn storybook:export && yarn demo:build && yarn demo:export"

[build.environment]
Copy link
Member Author

Choose a reason for hiding this comment

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

restored so we keep the version definition (help make sure it works in the lowest target we support)

Comment on lines +1 to +5
edge >= 83
firefox >= 77
chrome >= 83
safari >= 11
node 10.0
Copy link
Member Author

Choose a reason for hiding this comment

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

It looks like another advantage of not using TypeScipt to build the source but Babel. They don't seem to support browser target microsoft/TypeScript#19183.

@eps1lon
Copy link
Member

eps1lon commented Sep 26, 2020

It resonates with mui/material-ui#15496 from @eps1lon.

I'm definitely going to start with this the next week. Might not be the final target before release but it'll set up the final bundle structure and show in what direction the final bundle size goes.

@oliviertassinari
Copy link
Member Author

@oliviertassinari oliviertassinari marked this pull request as ready for review September 26, 2020 10:10
@oliviertassinari oliviertassinari merged commit 5d66e5c into mui:master Sep 28, 2020
@oliviertassinari oliviertassinari deleted the test-browser-stack branch September 28, 2020 10:40
dtassone pushed a commit to dtassone/material-ui-x that referenced this pull request Nov 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants