Skip to content

Mistakes in PeriodicFolderTrigger handling of ms-scale times#501

Merged
jtnord merged 1 commit intojenkinsci:masterfrom
jglick:PeriodicFolderTrigger
May 29, 2025
Merged

Mistakes in PeriodicFolderTrigger handling of ms-scale times#501
jtnord merged 1 commit intojenkinsci:masterfrom
jglick:PeriodicFolderTrigger

Conversation

@jglick
Copy link
Member

@jglick jglick commented May 29, 2025

#500 (comment) good catch @szjozsef though in fact this was using the wrong unit anyway, dating all the way back to jenkinsci/branch-api-plugin@5e85b53, and it does not seem to matter since the minimum permitted interval is 1m and that is also the minimum offered in the GUI dropdown.

@jglick jglick requested a review from a team as a code owner May 29, 2025 14:24
@jglick jglick added the bug label May 29, 2025
Copy link
Member

@jtnord jtnord left a comment

Choose a reason for hiding this comment

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

Yikes... the ability to perform a task every 1ms....

@jtnord jtnord enabled auto-merge May 29, 2025 14:27
@jtnord jtnord merged commit 4fcb721 into jenkinsci:master May 29, 2025
17 checks passed
@jglick jglick deleted the PeriodicFolderTrigger branch May 29, 2025 15:31
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.

2 participants