image-sequencer icon indicating copy to clipboard operation
image-sequencer copied to clipboard

Bump ora from 5.4.1 to 6.1.2

Open dependabot[bot] opened this issue 3 years ago • 4 comments

Bumps ora from 5.4.1 to 6.1.2.

Release notes

Sourced from ora's releases.

v6.1.2

https://github.com/sindresorhus/ora/compare/v6.1.1...v6.1.2

v6.1.1

  • Fix preserving stdin's pause state (#210) 77ccc1e

https://github.com/sindresorhus/ora/compare/v6.1.0...v6.1.1

v6.1.0

  • Expose the spinner interval as a getter 447812b
  • Internal refactoring. If you use any undocumented properties, this may break for you.

https://github.com/sindresorhus/ora/compare/v6.0.1...v6.1.0

v6.0.1

  • Upgrade dependencies 26ba606

https://github.com/sindresorhus/ora/compare/v6.0.0...v6.0.1

v6.0.0

Breaking

  • Require Node.js 12.20 ede1a54
  • This package is now pure ESM. Please read this.
  • ora.promise is now a named import called oraPromise (#181) 9c01990
    • import {oraPromise} from 'ora';
  • oraPromise now returns the given promise instead of this (#181) 9c01990
    • This lets you await it directly, which is more useful than chaining.

Improvements

  • Improve performance of the .clear() method (#182) d51c971

https://github.com/sindresorhus/ora/compare/v5.4.1...v6.0.0

Commits

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

dependabot[bot] avatar Jun 28 '22 07:06 dependabot[bot]

gitpod-io[bot] avatar Jun 28 '22 07:06 gitpod-io[bot]

Whoa that was a fast update.... anyways here were my notes from the previous:

ora.promise is now a named import called oraPromise ([#181](https://github-redirect.dependabot.com/sindresorhus/ora/issues/181)) 9c01990
import {oraPromise} from 'ora';
oraPromise now returns the given promise instead of this ([#181](https://github-redirect.dependabot.com/sindresorhus/ora/issues/181)) 9c01990
This lets you await it directly, which is more useful than chaining.

Referencing https://github.com/sindresorhus/ora/pull/181

Ora usages:

Looks like the first one is more relevant maybe... more use of asynchronous?

https://github.com/publiclab/image-sequencer/blob/3caa033c8febf5055c28f616b534933c00fcbe65/src/cli/sequencerSteps.js#L72-L87

https://github.com/publiclab/image-sequencer/blob/3caa033c8febf5055c28f616b534933c00fcbe65/src/cli/installModule.js#L11-L15

jywarren avatar Jun 28 '22 08:06 jywarren

Yes, it now returns a promise instead of the spinner object:

https://github.com/sindresorhus/ora/pull/181/files#diff-5a831ea67cf5cf8703b0de46901ab25bd191f56b320053be9332d9a3b0d01d15L229

So now we need to change starting on line 74 here, to receive spinnerPromise instead of spinnerObj, and to stop it differently on line 82-3?:

https://github.com/publiclab/image-sequencer/blob/3caa033c8febf5055c28f616b534933c00fcbe65/src/cli/sequencerSteps.js#L72-L87

jywarren avatar Jun 28 '22 09:06 jywarren

Oh you know what, the error is actually about the require() usage:

internal/modules/cjs/loader.js:1102
      throw new ERR_REQUIRE_ESM(filename, parentPath, packageJsonPath);
      ^
Error [ERR_REQUIRE_ESM]: Must use import to load ES Module: /home/runner/work/image-sequencer/image-sequencer/node_modules/ora/index.js
require() of ES modules is not supported.
require() of /home/runner/work/image-sequencer/image-sequencer/node_modules/ora/index.js from /home/runner/work/image-sequencer/image-sequencer/src/cli/installModule.js is an ES module file as it is a .js file whose nearest parent package.json contains "type": "module" which defines all .js files in that package scope as ES modules.
Instead rename index.js to end in .cjs, change the requiring code to use import(), or remove "type": "module" from /home/runner/work/image-sequencer/image-sequencer/node_modules/ora/package.json.

Maybe the promises stuff is not actually causing an issue... although we should check the spinner still stops properly.

The change is instead probably:

https://github.com/publiclab/image-sequencer/blob/3caa033c8febf5055c28f616b534933c00fcbe65/src/cli/installModule.js#L2

https://github.com/publiclab/image-sequencer/blob/3caa033c8febf5055c28f616b534933c00fcbe65/src/cli/sequencerSteps.js#L1

jywarren avatar Jun 28 '22 09:06 jywarren