Skip to content

Conversation

grodowski
Copy link
Contributor

Fixes #55708, please check the issue for detailed repro steps and context.

Checklist

Before submitting the PR make sure the following are checked:

  • This Pull Request is related to one change. Unrelated changes should be opened in separate PRs.
  • Commit message has a detailed description of what changed and why. If this PR fixes a related issue include it in the commit message. Ex: [Fix #issue-number]
  • Tests are added or updated if you fix a bug or add a feature.
  • CHANGELOG files are updated for the changed libraries if there is a behavior change or additional feature. Minor bug fixes and documentation changes should not be included.

@rafaelfranca rafaelfranca force-pushed the grodowski/file-update-checker-time-travel-fix branch from 615734d to 915062a Compare September 19, 2025 18:46
…hecker

Prevents uneccesary reloading in tests using time travel helpers
@rafaelfranca rafaelfranca force-pushed the grodowski/file-update-checker-time-travel-fix branch from 915062a to 321b340 Compare September 19, 2025 19:28
@rafaelfranca rafaelfranca merged commit a86199e into rails:main Sep 19, 2025
2 of 3 checks passed
@rafaelfranca rafaelfranca deleted the grodowski/file-update-checker-time-travel-fix branch September 19, 2025 20:11
rafaelfranca added a commit that referenced this pull request Sep 19, 2025
…time-travel-fix

[Fix #55708] Use process time instead of Time.now in FileUpdateChecker
rafaelfranca added a commit that referenced this pull request Sep 19, 2025
…time-travel-fix

[Fix #55708] Use process time instead of Time.now in FileUpdateChecker
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.

ActiveSupport::FileUpdateChecker triggers unnecessary reloads in tests using time travel

2 participants