Skip to content

Remove sudo from tee cmd as we're already in sudo cmd mode.#10291

Merged
rwestMSFT merged 2 commits intoMicrosoftDocs:livefrom
BeauSmyth19901:patch-1
Mar 5, 2026
Merged

Remove sudo from tee cmd as we're already in sudo cmd mode.#10291
rwestMSFT merged 2 commits intoMicrosoftDocs:livefrom
BeauSmyth19901:patch-1

Conversation

@BeauSmyth19901
Copy link
Contributor

@BeauSmyth19901 BeauSmyth19901 commented Mar 4, 2026

Just reading through the docs on this and caught what I think is a possible copy/paste mistake.
Since the steps begin with an invocation of sudo su I think that means we should be able to invoke tee without it until exit is invoked.

@prmerger-automator
Copy link
Contributor

@BeauSmyth19901 : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 60c3398:

✅ Validation status: passed

File Status Preview URL Details
docs/linux/includes/odbc-ubuntu.md ✅Succeeded

For more details, please refer to the build report.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the Ubuntu ODBC installation include to remove a redundant sudo from a tee command when the instructions have already switched into a root shell via sudo su.

Changes:

  • Remove sudo from the tee invocation used to install the Microsoft repository GPG key (Ubuntu 18.04 tab).

@ttorble
Copy link
Contributor

ttorble commented Mar 4, 2026

@rwestMSFT

Can you review the proposed changes?

IMPORTANT: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@prmerger-automator prmerger-automator bot added the aq-pr-triaged tracking label for the PR review team label Mar 4, 2026
remove missed redundant `sudo` calls.
@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit bd15f93:

✅ Validation status: passed

File Status Preview URL Details
docs/linux/includes/odbc-ubuntu.md ✅Succeeded

For more details, please refer to the build report.

@rwestMSFT rwestMSFT merged commit a0ef20b into MicrosoftDocs:live Mar 5, 2026
2 checks passed
@prmerger-automator
Copy link
Contributor

PR 10291 has been merged from BeauSmyth19901:patch-1 to MicrosoftDocs:live by rwestMSFT.

@BeauSmyth19901, @rwest

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants