Skip to content

Conversation

jonathanhefner
Copy link
Member

@jonathanhefner jonathanhefner commented Aug 5, 2023

When seeing "Last modified" time in the context of a version-controlled project, users are likely to think it is the time of the last revision. However, the "Last modified" time is the mtime of the file on disk, unrelated to the time of the last commit for the file or even for the branch as a whole.

Since that information can be misleading and is not particularly useful, this commit removes it.

When seeing "Last modified" time in the context of a version-controlled
project, users are likely to think it is the time of the last revision.
However, the "Last modified" time is the mtime of the file on disk,
unrelated to the time of the last commit for the file or even for the
branch as a whole.

Since that information can be misleading and is not particularly useful,
this commit removes it.
@jonathanhefner jonathanhefner force-pushed the banner-remove-modified-time-from-file branch from f11e642 to 67e3d7c Compare August 5, 2023 22:54
@jonathanhefner jonathanhefner merged commit f676ccc into rails:main Aug 5, 2023
rafaelfranca pushed a commit that referenced this pull request Sep 2, 2025
…me-from-file

Remove misleading "Last modified" time from `file.rhtml`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant