-
Notifications
You must be signed in to change notification settings - Fork 21.9k
functionTimeout adding format examples #127848
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
base: main
Are you sure you want to change the base?
Conversation
Added detailed explanation for functionTimeout format and usage.
|
@ffarinha-msft : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change. |
|
Learn Build status updates of commit 09735a4:
|
| File | Status | Preview URL | Details |
|---|---|---|---|
| articles/azure-functions/functions-host-json.md | Details |
articles/azure-functions/functions-host-json.md
- Line 328, Column 76: [Warning: hard-coded-locale - See documentation]
Link 'https://learn.microsoft.com/en-us/dotnet/fundamentals/runtime-libraries/system-timespan-parse' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites. - Line 328, Column 76: [Suggestion: docs-link-absolute - See documentation]
Absolute link 'https://learn.microsoft.com/en-us/dotnet/fundamentals/runtime-libraries/system-timespan-parse' will be broken in isolated environments. Replace with a relative link.
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.
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.
Pull Request Overview
This PR enhances the documentation for the functionTimeout configuration in Azure Functions by clarifying the valid timeout value syntax and providing concrete examples to prevent common formatting errors.
Key changes:
- Added a hyperlink to the official .NET TimeSpan parse documentation
- Documented the
[d.]hh:mm:ssformat syntax with explicit component definitions - Included a practical tip explaining how to properly set a 24-hour timeout
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Can you review the proposed changes? Important: When the changes are ready for publication, adding a #label:"aq-pr-triaged" |
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Diana Richards <v-dirichards@microsoft.com>
|
Learn Build status updates of commit 633ffbf: 💡 Validation status: suggestions
articles/azure-functions/functions-host-json.md
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 status updates of commit 1cbdbc4: 💡 Validation status: suggestions
articles/azure-functions/functions-host-json.md
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. |
The following PR aims to provide clarity on the valid timeout values syntax that can be defined to avoid confusion.
As an example to set a 24 hours timeout one might think you need to set "24:00:00" but this is not accurate instead you need to use
"1.00:00:00"or"23:59:59"