Skip to content

Conversation

@simonsabin
Copy link
Contributor

@simonsabin simonsabin commented Nov 5, 2025

Point people at GREATEST OELEAST if looking for max of a set of values inline

Point people at GREATEST if looking for max of a set of values inline
@prmerger-automator
Copy link
Contributor

@simonsabin : 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 c1d8b70:

✅ Validation status: passed

File Status Preview URL Details
docs/t-sql/functions/max-transact-sql.md ✅Succeeded

For more details, please refer to the build report.

Corrected grammatical errors for clarity.
Updated the description of the MIN function to clarify its behavior and added a reference to the LEAST function for inline minimum values.
@simonsabin simonsabin marked this pull request as ready for review November 5, 2025 14:20
@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit b602f3d:

⚠️ Validation status: warnings

File Status Preview URL Details
docs/t-sql/functions/max-transact-sql.md ⚠️Warning Details

docs/t-sql/functions/max-transact-sql.md

  • Line 24, Column 100: [Warning: file-not-found - See documentation] Invalid file link: '../../t-sql/queries/select-over-clause-transact-sql'.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 9e66ea8:

⚠️ Validation status: warnings

File Status Preview URL Details
docs/t-sql/functions/max-transact-sql.md ⚠️Warning Details
docs/t-sql/functions/min-transact-sql.md ⚠️Warning Details

docs/t-sql/functions/max-transact-sql.md

  • Line 24, Column 100: [Warning: file-not-found - See documentation] Invalid file link: '../../t-sql/queries/select-over-clause-transact-sql'.

docs/t-sql/functions/min-transact-sql.md

  • Line 24, Column 100: [Warning: file-not-found - See documentation] Invalid file link: '../../t-sql/queries/select-over-clause-transact-sql'.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit a263ba9:

⚠️ Validation status: warnings

File Status Preview URL Details
docs/t-sql/functions/max-transact-sql.md ⚠️Warning Details
docs/t-sql/functions/min-transact-sql.md ✅Succeeded

docs/t-sql/functions/max-transact-sql.md

  • Line 24, Column 100: [Warning: file-not-found - See documentation] Invalid file link: '../../t-sql/queries/select-over-clause-transact-sql'.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit caeb67e:

✅ Validation status: passed

File Status Preview URL Details
docs/t-sql/functions/max-transact-sql.md ✅Succeeded
docs/t-sql/functions/min-transact-sql.md ✅Succeeded

For more details, please refer to the build report.

@ktoliver
Copy link
Contributor

ktoliver commented Nov 5, 2025

@markingmyname

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 Nov 5, 2025
@rwestMSFT
Copy link
Contributor

#reassign:rwestMSFT

@rwestMSFT
Copy link
Contributor

Thanks, Simon. We'll do an internal PR to merge this. We're up against a content freeze for an upcoming conference.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants