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

Tests: Update regexes and include MR fix #1296

Merged
merged 4 commits into from
Mar 2, 2021
Merged

Conversation

Cawllec
Copy link
Contributor

@Cawllec Cawllec commented Feb 16, 2021

Goal

Updates the regex's added in the react-native CLI maze-runner update #1293. In addition updates the version of Maze-runner required to apply latest fixes.

@github-actions
Copy link

github-actions bot commented Feb 16, 2021

@bugsnag/browser bundle size diff

Minified Minfied + Gzipped
Before 40.76 kB 12.57 kB
After 40.76 kB 12.57 kB
± No change No change

code coverage diff

Coverage values did not change👌.

Total:

Lines Branches Functions Statements
0%(+0%) 0%(+0%) 0%(+0%) 0%(+0%)

Generated by 🚫 dangerJS against a55b0ef

When I input a return interactively
Then I wait for the shell to output a match for the regex "@bugsnag/source-maps dependency is installed" to stdout
Then I wait for the shell to output a match for the regex "@bugsnag\/source-maps dependency is installed" to stdout
Copy link
Contributor

Choose a reason for hiding this comment

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

Sorry to be annoying, but these forward slashes don't actually need to be escaped because of how the regex is constructed, e.g.

irb(main):001:0> a = Regexp.new("http:\/\/example\.com")
irb(main):002:0> a.match? "http://example.com"
=> true
irb(main):003:0> b = Regexp.new("http://example\.com")
irb(main):004:0> b.match? "http://example.com"
=> true

The URLs especially get a bit hard to read with the escaping

Copy link
Contributor

@twometresteve twometresteve 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 happy if Joe is.

@twometresteve twometresteve force-pushed the tests/rn-cli-mr-update branch from 13d6032 to a55b0ef Compare March 2, 2021 09:37
@twometresteve twometresteve merged commit ae4a017 into next Mar 2, 2021
@twometresteve twometresteve deleted the tests/rn-cli-mr-update branch March 2, 2021 10:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants