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

chore(deps): update dependency open to v7 (next) #2282

Closed
wants to merge 22 commits into from

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented Oct 15, 2019

This PR contains the following updates:

Package Type Update Change
open dependencies major ^6.2.0 -> ^7.0.0

Release Notes

sindresorhus/open

v7.0.0

Compare Source

Breaking
  • Correctly handle URL and path escaping on Windows (#​146) 7ef15d2
    If you use open with URLs, you'll want to use the new {url: true} option.
    Read more.
Enhancements

v6.4.0

Compare Source

v6.3.0

Compare Source

  • Use the system xdg-open on Linux when open is bundled/packaged (#​134) c69fd06

Renovate configuration

📅 Schedule: At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

♻️ Rebasing: Whenever PR becomes conflicted, or if you modify the PR title to begin with "rebase!".

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box

Newsflash: Renovate has joined WhiteSource, and is now free for all use. Learn more or view updated terms and privacy policies.

hiroppy and others added 22 commits September 10, 2019 07:52
Because opn was renamed to open.
* feat(server): add stdin for api

* test(stdin): switch to async await tests for stdin

* test(cli): use await timer
* feat(server): made user callback async, remove cli listen redundancy

* test(server): added server listen method async callback tests

* test(server): create options object using spread operator

* chore(server): fix comment spelling
* refactor: update snapshot & change clientSocketOptions type to Object

* refactor: union sockHost, sockPath and sockPort in clientSocketOptions

* refactor: remove console.error
@codecov
Copy link

codecov bot commented Oct 15, 2019

Codecov Report

Merging #2282 into next will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             next    #2282   +/-   ##
=======================================
  Coverage   93.93%   93.93%           
=======================================
  Files          36       36           
  Lines        1302     1302           
  Branches      367      366    -1     
=======================================
  Hits         1223     1223           
- Misses         70       78    +8     
+ Partials        9        1    -8
Impacted Files Coverage Δ
client-src/clients/WebsocketClient.js 58.97% <0%> (ø) ⬆️
client-src/clients/BaseClient.js 69.23% <0%> (ø) ⬆️
client-src/clients/SockJSClient.js 60% <0%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update adc1154...14a91c7. Read the comment docs.

@renovate
Copy link
Contributor Author

renovate bot commented Dec 12, 2019

PR has been edited

👷 This PR has received other commits, so Renovate will stop updating it to avoid conflicts or other problems. If you wish to abandon your changes and have Renovate start over you may click the "rebase" checkbox in the PR body/description.

hiroppy added a commit that referenced this pull request Dec 20, 2019
@hiroppy hiroppy closed this Dec 20, 2019
@hiroppy hiroppy deleted the renovate/next-open-7.x branch December 20, 2019 09:47
@renovate
Copy link
Contributor Author

renovate bot commented Dec 20, 2019

Autoclosing Skipped

This PR has been flagged for autoclosing, however it is being skipped due to the branch being already modified. Please close/delete it manually or report a bug if you think this is in error.

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.

4 participants