-
-
Notifications
You must be signed in to change notification settings - Fork 45
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
Feature request: exclude specific commits when calculating the 'last update' calculation #113
Comments
That would be a nice addition. I'd prefer to this outside of the plugin if possible. For example, for Preferably I would like a similar approach for this plugin (which uses mkdocs-git-revision-date-localized-plugin/mkdocs_git_revision_date_localized_plugin/util.py Lines 131 to 134 in 6150fad
Do you already have a specific approach in mind? |
The Going through this blogpost, it seems we could add a Update: from the blogpost, it also requires setting a global or local git config option (something like If that works, we would only need to update the documentation with an additional 'how to' page. |
Thanks for the quick reply! I played around with the options (including the one you suggested), but didn't find anything that would work. The closest I could get is to exclude certain commits by commit message, but even this requires cli parameters and doesn't seem to be configurable via the git config:
Meanwhile my colleague implemented this feature and will send a PR after testing it. |
In our usage it sometimes happens that we need to make changes to many/all markdown files at once, e.g. running a formatter, or when the name of an internal product or team changes. We would like to exclude these changes from the 'last update' calculation, as they don't mean that the content was actually checked and actualized (but merely that someone run a search and replace over the entire repo). This doesn't happen that often, so we have a limited number of these commits (less than 5).
Thus I'd like to propose a new config option called
exclude_commits
as a list of commit ids that should not be considered when calculating the last update time, e.g.If this sounds like something you would accept to be part of the plugin, I'm happy to send a PR with the implementation and doc update.
The text was updated successfully, but these errors were encountered: