Fix WEEK_CEIL 2-week example output #3251
Conversation
There was a problem hiding this comment.
Pull request overview
This PR fixes an incorrect example output in the documentation for the WEEK_CEIL SQL function. The example demonstrates the function with a 2-week period parameter, and the previous output incorrectly showed rounding to the next 1-week boundary instead of the correct 2-week boundary.
Key Changes:
- Corrected the example output date from
2023-07-17 00:00:00to2023-07-24 00:00:00for theWEEK_CEIL('2023-07-13 22:28:18', 2)example - Applied the fix consistently across all language versions (English and Chinese) and documentation versions (2.1 and 3.x)
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| versioned_docs/version-3.x/sql-manual/sql-functions/scalar-functions/date-time-functions/week-ceil.md | Fixed the 2-week example output in the English version 3.x documentation |
| versioned_docs/version-2.1/sql-manual/sql-functions/scalar-functions/date-time-functions/week-ceil.md | Fixed the 2-week example output in the English version 2.1 documentation |
| i18n/zh-CN/docusaurus-plugin-content-docs/version-3.x/sql-manual/sql-functions/scalar-functions/date-time-functions/week-ceil.md | Fixed the 2-week example output in the Chinese version 3.x documentation |
| i18n/zh-CN/docusaurus-plugin-content-docs/version-2.1/sql-manual/sql-functions/scalar-functions/date-time-functions/week-ceil.md | Fixed the 2-week example output in the Chinese version 2.1 documentation |
The changes look good. The correction is mathematically accurate (when using a 2-week period, the function should round up to the nearest 2-week boundary, not just the next week), and the fix has been applied consistently across all documentation versions and languages. No issues were found with the changes.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
1 participant
Versions
Languages
Docs Checklist