Document the AddTimeSpan date filter - #3328
Closed
NickJosevski wants to merge 1 commit into
Closed
Conversation
Spells out the {days}.{hours}:{minutes}:{seconds} format, links to the
.NET TimeSpan.Parse docs, and calls out that 48:00:00 is 48 days rather
than 48 hours because the leading field stops meaning hours above 23.
Also fixes pre-existing markdownlint violations in this file, which CI
surfaces because it lints only changed files. Table content is unchanged,
only cell padding and separator widths.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
|
Pull request environment is available at https://stoctodocspr3328.z22.web.core.windows.net. You can view the ephemeral environment status in Octopus Deploy. This environment will be automatically deprovisioned when the pull request is closed, or after 7 days of inactivity. |
This was referenced Aug 10, 2026
Contributor
Author
|
Closing — going with the |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Variant of #3326, documenting the single-filter shape from OctopusDeploy/Octostache#125 instead of separate
AddHours/AddDays.Changes
variable-filters.md—AddTimeSpanrow in the date-filters table, plus an### AddTimeSpansection.variable-substitutions.mdx— added to the flat filter list.Emphasis
The format is stated explicitly as
{days}.{hours}:{minutes}:{seconds}in a code block, and linked to the .NET TimeSpan.Parse docs.A warning callout covers the trap directly —
48:00:00is 48 days, not 48 hours — with a table showing the23:00:00→24:00:00flip and the fact that24:01:02reads02as minutes rather than seconds. It ends by pointing at the correct way to say 48 hours:"2.00:00:00".A second callout covers quoting: the day form contains a
., which isn't valid in an unquoted filter argument.Worked examples include 2 days (
"2.00:00:00") and 2.5 days ("2.12:00:00", ie 60 hours), both with the resulting output.The ServiceNow change-window example carries over, along with the warning that
Octopus.Task.QueueTimeis queue time rather than execution start.Second commit
Fixes pre-existing markdownlint violations in
variable-filters.md— misaligned table pipes, asterisk list markers, hard tabs. These already fail onmain; CI lints only changed files, so touching the page surfaces them. Table content is unchanged, only padding.Do not merge before OctopusDeploy/Octostache#125 ships.
🤖 Generated with Claude Code