Skip to content

[rush-lib] Supports the rush install-autoinstaller command #4823

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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 21 commits into from
Aug 21, 2024
Merged

[rush-lib] Supports the rush install-autoinstaller command #4823

merged 21 commits into from
Aug 21, 2024

Conversation

L-Qun
Copy link
Contributor

@L-Qun L-Qun commented Jul 4, 2024

Summary

  • Supports the rush install-autoinstaller command to prevent accidental updates to the lockfile.

  • Exports Autoinstaller and RushGlobalFolder class for easy use in business scenarios leveraging the rush-sdk.

How it was tested

Manually tested with Rushstack repo locally.

Impacted documentation

microsoft/rushstack-websites#233

@L-Qun L-Qun changed the title feat: support install-autoinstaller [rush-lib] feat: support install-autoinstaller Jul 4, 2024
@L-Qun L-Qun changed the title [rush-lib] feat: support install-autoinstaller [rush-lib] support install-autoinstaller Jul 4, 2024
@L-Qun L-Qun changed the title [rush-lib] support install-autoinstaller [rush-lib] Supports the rush install-autoinstaller command Jul 5, 2024
@L-Qun L-Qun requested a review from patmill as a code owner July 11, 2024 01:25
@L-Qun L-Qun requested a review from iclanton July 11, 2024 05:15
@L-Qun L-Qun requested a review from iclanton July 16, 2024 10:30
@L-Qun
Copy link
Contributor Author

L-Qun commented Jul 17, 2024

Thanks to @iclanton, I learned a lot about coding practices!

@L-Qun
Copy link
Contributor Author

L-Qun commented Jul 24, 2024

Hi @iclanton, all your comments have been resolved, can you take another look at this PR?

Copy link
Member

@iclanton iclanton left a comment

Choose a reason for hiding this comment

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

@L-Qun - I left a few comments. We can probably get this out soon.

@iclanton iclanton enabled auto-merge (squash) August 21, 2024 15:31
@iclanton iclanton merged commit 0a502f0 into microsoft:main Aug 21, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Closed
Development

Successfully merging this pull request may close these issues.

2 participants