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

docs: add explicit PLACEHOLDER and TODO markers in the readme #244

Merged
merged 1 commit into from
Aug 15, 2022

Conversation

nedbat
Copy link
Contributor

@nedbat nedbat commented Aug 12, 2022

This will serve as clear indicators:

  • Readers will know the content is auto-generated
  • Maintainers will know the content needs to be updated
  • Maintainers will know where edits are needed
  • Linters can find these markers to alert people that work is needed.

Copy link
Contributor

@feanil feanil left a comment

Choose a reason for hiding this comment

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

Makes sense to me. :shipit:

Comment on lines +11 to +13
.. I wish we used a tool to manage changelogs so that I didn't have to choose
a date when I made a commit, since I don't know what date the change will
be merged...
Copy link
Contributor

Choose a reason for hiding this comment

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

And we don't even "release" cookiecutters. I have a canned rant about how we manage versioning in changelogs for repos where we do cut releases...

This will serve as clear indicators:

- Readers will know the content is auto-generated
- Maintainers will know the content needs to be updated
- Maintainers will know where edits are needed
- Linters can find these markers to alert people that work is needed.
@nedbat nedbat force-pushed the nedbat/readme-placeholders branch from 93729bd to 232d980 Compare August 15, 2022 17:26
@nedbat nedbat merged commit 0ddc3ce into master Aug 15, 2022
@nedbat nedbat deleted the nedbat/readme-placeholders branch August 15, 2022 17:33
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.

3 participants