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

update browserstack browser list as some devices have been removed such as the Samsung Galaxy Note 4 that we tested on #1108

Merged
merged 2 commits into from
Oct 18, 2021

Conversation

JakeChampion
Copy link
Owner

@JakeChampion JakeChampion commented Oct 18, 2021

On 06 October 2021 BrowserStack deprecated and removed the Samsung Galaxy Note 4-4.4

This pull-request updates our device list to the latest devices and browsers that BrowserStack support

Other pull-requests will be blocked until we update our device list to not include Samsung Galaxy Note 4-4.4

…oved such as the Samsung Galaxy Note 4 that we tested on
@JakeChampion JakeChampion requested a review from a team as a code owner October 18, 2021 14:15
@JakeChampion JakeChampion added the current Used to mark an issue as planned for Origami's current six-week cycle label Oct 18, 2021
@JakeChampion JakeChampion added this to active in Origami ✨ via automation Oct 18, 2021
@JakeChampion JakeChampion moved this from active to ready to review in Origami ✨ Oct 18, 2021
@github-actions github-actions bot added the library Relates to an Origami library label Oct 18, 2021
Copy link
Collaborator

@chee chee left a comment

Choose a reason for hiding this comment

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

i'm not sure how to review this 👍

@JakeChampion
Copy link
Owner Author

@chee I've added some documentation to the contributing.md file about the files and their use. I've also added a comment to the top of both files to mention they are generated via a script and what command to use to regenerate them

Comment on lines +158 to +171
## Updating the list of browsers used during testing on BrowserStack

The files `test/polyfills/browsers.toml` and `test/polyfills/browserstackBrowsers.toml` are used to map the browser names used in polyfill configuration files to their corresponding record on BrowserStack.

To update those files with the latest set of browsers that BrowserStack support, you should run:

`$ npm run update-browserstack-list`

If the command is successful then your terminal will respond:

```
Updated the browser list for automated testing via BrowserStack.
```

Copy link
Collaborator

Choose a reason for hiding this comment

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

very nice :)

@JakeChampion JakeChampion merged commit 65b6a73 into master Oct 18, 2021
Origami ✨ automation moved this from ready to review to complete Oct 18, 2021
@JakeChampion JakeChampion deleted the devices branch October 18, 2021 16:40
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 19, 2022
@robertboulton robertboulton removed this from Done in Origami ✨ Jul 26, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
current Used to mark an issue as planned for Origami's current six-week cycle library Relates to an Origami library
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants