-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
#3718: Added date to changelog #3788
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work!
Thank You @humitos :) :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Noted a more proper reST use
contrib/changelog.hbs
Outdated
@@ -1,5 +1,7 @@ | |||
{{header}} | |||
{{toHeader header "-"}} | |||
|
|||
Date: {{date}} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be an reST fied list:
http://docutils.sourceforge.net/docs/user/rst/quickref.html#field-lists
For example, note the difference in display here:
http://rst.ninjs.org/#RGF0ZTogTWFyY2ggMjMsIDIwMTgKCjpEYXRlOiBNYXIgMjMsIDIwMTg=
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @agjohnson , I have made the changes. Do guide me if anything more has to be done :)
Hi @agjohnson just wanted to know the status of the PR. Is there anything to be updated from my side? |
@rajujha373 probably not, review just takes some time on our end. |
Thank you :) @agjohnson 🎉 |
This PR is in reference to PR #3727
@humitos @stsewd Please have a look to the changes made here