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.

Sign up for GitHub

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

Jump to bottom

feat(cli): use inner npminstall、use default registry mirror #71

Merged
merged 1 commit into from
Jul 18, 2023
Merged

feat(cli): use inner npminstall、use default registry mirror #71

merged 1 commit into from
Jul 18, 2023

Conversation

hexqi
Copy link
Collaborator

@hexqi hexqi commented Jul 4, 2023

PR

PR Checklist

Please check if your PR fulfills the following requirements:

  • The commit message follows our Commit Message Guidelines
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • Other... Please describe:

What is the current behavior?

Install command: npm i @opentiny/cli npminstall -g

Issue Number: N/A

What is the new behavior?

Install command: npm i @opentiny/cli -g

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

@hexqi hexqi requested review from hugohua and h-ivy July 4, 2023 04:10
@hexqi hexqi force-pushed the feat/inner-installer branch from 0ac5a32 to 96fc2ed Compare July 4, 2023 07:26
h-ivy
h-ivy approved these changes Jul 18, 2023
View reviewed changes
Copy link
Contributor

@h-ivy h-ivy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok

@h-ivy h-ivy merged commit 8c8a772 into opentiny:dev Jul 18, 2023
@kagol kagol added the cli cli label Apr 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@h-ivy h-ivy h-ivy approved these changes

@hugohua hugohua Awaiting requested review from hugohua

Assignees
No one assigned
Labels
cli cli
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@hexqi @h-ivy @kagol