Skip to content

Add /norestart in the command for installing #2065

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

Closed
wants to merge 2 commits into from
Closed

Add /norestart in the command for installing #2065

wants to merge 2 commits into from

Conversation

peterneave
Copy link

Make users (like me) aware that the /quiet command will reboot the machine without intervention

Make users (like me) aware that the /quiet command will reboot the machine without intervention
@saldana
Copy link
Contributor

saldana commented Jan 17, 2018

✅ Validation status: passed

File Status Preview URL Details
wmf/5.1/install-configure.md ✅Succeeded

For more details, please refer to the build report.

Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

@exchange12rocks
Copy link
Contributor

For consistency, I suggest you to add those parameters into wmf/5.0/install.md as well. You could do this just by implementing changes in the "peterneave:patch-1" branch - they will be automatically added to this pull-request.

Make users (like me) aware that the /quiet command will reboot the machine without intervention
@msftclas
Copy link

msftclas commented Jan 17, 2018

CLA assistant check
All CLA requirements met.

@peterneave
Copy link
Author

As requested @exchange12rocks

@saldana
Copy link
Contributor

saldana commented Jan 17, 2018

✅ Validation status: passed

File Status Preview URL Details
wmf/5.0/install.md ✅Succeeded
wmf/5.1/install-configure.md ✅Succeeded

For more details, please refer to the build report.

Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

@sdwheeler
Copy link
Contributor

@zjalexander @joeyaiello - What do you think about this change? Do we need to add a reboot step?

@sdwheeler
Copy link
Contributor

@peterneave Thanks for your contribution. I feel your pain about the reboot. But it is necessary. I talked this over with Zach. We will update the article to talk about the reboot. But I won't take this PR as is. I will have an alternate update to this article by tomorrow.

@sdwheeler sdwheeler closed this Jan 19, 2018
@sdwheeler sdwheeler mentioned this pull request Jan 19, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants