Skip to content
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] Drop CVS $Id$ stuff #286

Open
BenBE opened this issue Apr 5, 2023 · 0 comments
Open

[FEATURE] Drop CVS $Id$ stuff #286

BenBE opened this issue Apr 5, 2023 · 0 comments

Comments

@BenBE
Copy link

BenBE commented Apr 5, 2023

Is your feature request related to a problem? Please describe.
When looking at the code I noticed that it contains $Id: $ comments from the CVS days. As the source is currently managed in Git those markers are likely unused (unless Git has been set up to do content transforms, which is non-default behaviour).

Describe the solution you'd like
Thus I'd suggest to globally remove those $Id: $ content transform comments from the code base.

Describe alternatives you've considered
None. Leaving things as is will just lead to those comments staying untouched and thus out-of-date eventually.

Additional context
Cf. the section keyword expansion in the Git SCM manual

There has been some discussions of using keyword substitution with Git (e.g. linked in the FAQ, most of it advising against it.

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

No branches or pull requests

1 participant