Skip to content

Commit

Permalink
Chore: Move react-native-url-polyfill to dev dependencies (#34687)
Browse files Browse the repository at this point in the history
* Move react-native-url-polyfill to devDep in @wordpress/url

No longer necessary as a runtime dependency after is-url.native.js was removed

* Move react-native-url-polyfill to dev dependencies of the project

Co-authored-by: Ben Lowery <blowery@automattic.com>
  • Loading branch information
gziolo and blowery committed Sep 9, 2021
1 parent 1a1a9a0 commit 0c47198
Show file tree
Hide file tree
Showing 4 changed files with 7 additions and 4 deletions.
3 changes: 1 addition & 2 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

1 change: 1 addition & 0 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -196,6 +196,7 @@
"react": "17.0.1",
"react-dom": "17.0.1",
"react-native": "0.64.0",
"react-native-url-polyfill": "1.1.2",
"react-test-renderer": "17.0.1",
"rimraf": "3.0.2",
"rtlcss": "2.6.2",
Expand Down
4 changes: 4 additions & 0 deletions packages/url/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,10 @@

## Unreleased

### Bug Fix

- Removed unused `react-native-url-polyfill` dependency ([#34687](https://github.com/WordPress/gutenberg/pull/34687)).

## 3.2.0 (2021-07-21)

## 3.1.0 (2021-05-20)
Expand Down
3 changes: 1 addition & 2 deletions packages/url/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -28,8 +28,7 @@
"sideEffects": false,
"dependencies": {
"@babel/runtime": "^7.13.10",
"lodash": "^4.17.21",
"react-native-url-polyfill": "^1.1.2"
"lodash": "^4.17.21"
},
"publishConfig": {
"access": "public"
Expand Down

0 comments on commit 0c47198

Please sign in to comment.