Skip to content

Hardcode mrm@2 in the documentation - #976

Merged
okonet merged 1 commit into
lint-staged:masterfrom
Palid:patch-1
May 10, 2021
Merged

okonet merged 1 commit into
lint-staged:masterfrom
Palid:patch-1

Conversation

@Palid

@Palid Palid commented May 10, 2021

Copy link
Copy Markdown
Contributor

Due to some changed in mrm@3 the entire installation task fails without easily searchable solution. This fixes the problem by hard locking mrm to @2 for as long as it's not solved properly.

Due to some changed in mrm@3 the entire installation task fails without easily searchable solution. This fixes the problem by hard locking mrm to @2 for as long as it's not solved properly.
@Palid

Palid commented May 10, 2021

Copy link
Copy Markdown
Contributor Author

Example of the issue:

❯ npx mrm lint-staged
npx: installed 298 in 9.141s

Preset “default” not found.

We’ve tried to load “mrm-preset-default” and “default” npm packages.

@iiroj

iiroj commented May 10, 2021

Copy link
Copy Markdown
Member

Thanks @Palid!

@iiroj

iiroj commented May 10, 2021

Copy link
Copy Markdown
Member

I believe this fixes #970

@codecov

codecov Bot commented May 10, 2021

Copy link
Copy Markdown

Codecov Report

Merging #976 (c31df34) into master (e5e186d) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##            master      #976   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           18        18           
  Lines          611       611           
  Branches       144       144           
=========================================
  Hits           611       611           

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 e5e186d...c31df34. Read the comment docs.

@okonet

okonet commented May 10, 2021

Copy link
Copy Markdown
Collaborator

Hmm I wonder if that’s the best way. @sapegin could you please take a look at why it’s not working anymore?

@Palid

Palid commented May 10, 2021

Copy link
Copy Markdown
Contributor Author

@okonet I think that it's not a best way to do it, but it's better to have this temporary fix for documentation and figure out what's wrong there instead of having it broken without any simple solution. IMHO the best way to fix it now would be to merge this and revert it with the proper fix later. I can try investigating more on Wednesday too.

@okonet

okonet commented May 10, 2021

Copy link
Copy Markdown
Collaborator

I agree! Thanks for it. Let’s merge it.

@okonet
okonet merged commit 4f9a146 into lint-staged:master May 10, 2021
@iiroj iiroj linked an issue May 11, 2021 that may be closed by this pull request
@Palid

Palid commented May 19, 2021

Copy link
Copy Markdown
Contributor Author

@okonet seems like a fix is waiting in the merge queue for a while now: sapegin/mrm#163

@github-actions

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 11.0.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

zeekay pushed a commit to hanzoai/lint-staged that referenced this pull request Sep 9, 2026
Due to some changed in mrm@3 the entire installation task fails without easily searchable solution. This fixes the problem by hard locking mrm to @2 for as long as it's not solved properly.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

docs: mrm-task-lint-staged has to be installed

3 participants