Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

feat(init): add pnpm support#915

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
jimthedev merged 1 commit intocommitizen:masterfromchenaski:master
Jan 19, 2023
Merged

feat(init): add pnpm support#915

jimthedev merged 1 commit intocommitizen:masterfromchenaski:master
Jan 19, 2023

Conversation

@chenaski
Copy link
Contributor

@chenaskichenaski commentedMar 25, 2022•
edited
Loading

Added--pnpm option. Forpnpm available--save,--save-dev and--save-exact options.

Resolves:#893,resolves#858,resolves#884.

Jogiter, hellorayza, idmyn, the-ult, EdwardKerckhof, hexagon141, CRBroughton, byeze, mheob, memory-lovers, and 8 more reacted with thumbs up emojiJogiter, hellorayza, ysya, mosu1314, hexagon141, mitchelvanbever, and howard86 reacted with eyes emoji
pnpm reuses options available for npm, except `force`re:#893, re#858
@huzedong2015
Copy link

How much longer will it take?

hellorayza reacted with eyes emoji

@chenaski
Copy link
ContributorAuthor

Are there any extra steps I should take?

hellorayza, ysya, and karinfam reacted with eyes emoji

@hellorayza
Copy link

Can anyone help?

@lisen6
Copy link

Why doesn't mine work

@pulanski
Copy link

would love to see these changes added

1 similar comment
@jiamianmao
Copy link

would love to see these changes added

@mattyaodoit
Copy link

Keen to get this merged and supportpnpm

@enrique-ramirez
Copy link

Any news on this?

@chenaski
Copy link
ContributorAuthor

Does anyone know how everything works in this repo?

@HubooDeclan
Copy link

Can we get this merged?

mheob and AmmarArnt reacted with eyes emoji

@AmmarArnt
Copy link

Would love this PR merged too. Is this repo maintained?

@Ruobei
Copy link

would love to see these changes added

@wangrongding
Copy link

Looking forward to this pr being merged. 👀

@jimthedevjimthedev merged commitc1f4142 intocommitizen:masterJan 19, 2023
@github-actions
Copy link

🎉 This PR is included in version 4.3.0 🎉

The release is available on:

Yoursemantic-release bot 📦🚀

@jimthedev
Copy link
Member

It released but thetests are failing. So

  1. Build should not allow this.
  2. We may need to roll back or at least mark this as breaking and fix the tests.
chenaski reacted with eyes emoji

@chenaski
Copy link
ContributorAuthor

I don't think there are any breaking changes, it seems I just forgot to update azure pipeline.
We need to add apnpm installation before running the tests.

@Jalever
Copy link

Great!

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@jimthedevjimthedevjimthedev approved these changes

+1 more reviewer
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

feature request: add pnpm flag to the commitizen init flag Initialization error commitizen init doesn't work with pnpm

14 participants

@chenaski@huzedong2015@hellorayza@lisen6@pulanski@jiamianmao@mattyaodoit@enrique-ramirez@HubooDeclan@AmmarArnt@Ruobei@wangrongding@jimthedev@Jalever
[8]ページ先頭
©2009-2025 Movatter.jp