-
Notifications
You must be signed in to change notification settings - Fork 143
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
feat(microsoft_store): Add Microsoft Store step for Windows #963
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The spanish translations looks great! Nothing to change.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks for the implementation!
locales/app.yml
Outdated
"Microsoft Store": | ||
en: "Microsoft Store" | ||
es: "Tienda de Microsoft" | ||
zh_TW: "Microsoft 店鋪" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just checked the zh_TW translations and left a comment, let's merge the PR once we have it resolved:) BTW, there is a conflict in |
Add Microsoft Store Apps update step for Windows as Winget cannot update all Microsoft Store apps yet. Closes topgrade-rs#912
Thanks @SteveLauC @olivertzeng, should be ready to merge now then! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
Add Microsoft Store Apps update step for Windows as Winget cannot update all Microsoft Store apps yet.
Closes #912.
What does this PR do
Adds a Microsoft Store Apps update step for Windows.
Standards checklist
CONTRIBUTING.md
For new steps
--dry-run
option works with this step--yes
option works with this step if it is supported bythe underlying command
If you developed a feature or a bug fix for someone else and you do not have the
means to test it, please tag this person here.
@Rikiub, could you verify the Spanish translations?
@olivertzeng, could you verify the Taiwanese translations?
Success
Error
Note that the '0' in output will be a number other than 0 when a real error occurs.