intake / intake-stac

Intake interface to STAC data catalogs
https://intake-stac.readthedocs.io/en/latest/
BSD 2-Clause "Simplified" License
109 stars 26 forks source link

Update prettier pre-commit version #94

Closed TomAugspurger closed 3 years ago

TomAugspurger commented 3 years ago

This fixes an install issue with the old pinned version of prettier

[INFO] Installing environment for https://github.com/prettier/prettier.
[INFO] Once installed this environment will be reused.
[INFO] This may take a few minutes...
An unexpected error has occurred: CalledProcessError: command: ('/home/taugspurger/.cache/pre-commit/repoxw7opgyr/node_env-system/bin/node', '/usr/local/bin/npm', 'install', '--dev', '--prod', '--ignore-prepublish', '--no-progress', '--no-save')
return code: 1
expected return code: 0
stdout: (none)
stderr:
    npm WARN install Usage of the `--dev` option is deprecated. Use `--include=dev` instead.
    npm notice
    npm notice New minor version of npm available! 7.0.15 -> 7.18.1
    npm notice Changelog: <https://github.com/npm/cli/releases/tag/v7.18.1>
    npm notice Run `npm install -g npm@7.18.1` to update!
    npm notice
    npm ERR! code ERESOLVE
    npm ERR! ERESOLVE unable to resolve dependency tree
    npm ERR!
    npm ERR! While resolving: prettier@2.0.5
    npm ERR! Found: rollup@2.3.2
    npm ERR! node_modules/rollup
    npm ERR!   dev rollup@"2.3.2" from the root project
    npm ERR!
    npm ERR! Could not resolve dependency:
    npm ERR! peer rollup@"^1.20.0" from @rollup/plugin-alias@3.0.1
    npm ERR! node_modules/@rollup/plugin-alias
    npm ERR!   dev @rollup/plugin-alias@"3.0.1" from the root project
    npm ERR!
    npm ERR! Fix the upstream dependency conflict, or retry
    npm ERR! this command with --force, or --legacy-peer-deps
    npm ERR! to accept an incorrect (and potentially broken) dependency resolution.
    npm ERR!
    npm ERR! See /home/taugspurger/.npm/eresolve-report.txt for a full report.

    npm ERR! A complete log of this run can be found in:
    npm ERR!     /home/taugspurger/.npm/_logs/2021-06-20T14_56_18_844Z-debug.log

The second commit d1d40f301a92d519e93781be15913ca49b7bbd51 has reformatted the files with the new version.

TomAugspurger commented 3 years ago

CI failure is from upstream, which I'm working on now, so this should be good to merge.

cc @jhamman @scottyhq

jhamman commented 3 years ago

Thanks @TomAugspurger