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

#3718: Added date to changelog #3788

Merged
merged 2 commits into from
Mar 29, 2018
Merged

#3718: Added date to changelog #3788

merged 2 commits into from
Mar 29, 2018

Conversation

irkartik
Copy link
Contributor

This PR is in reference to PR #3727
@humitos @stsewd Please have a look to the changes made here
screenshot from 2018-03-14 00-29-37

Copy link
Member

@humitos humitos left a comment

Choose a reason for hiding this comment

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

Nice work!

@irkartik
Copy link
Contributor Author

Thank You @humitos :) :)

Copy link
Contributor

@agjohnson agjohnson left a 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

@@ -1,5 +1,7 @@
{{header}}
{{toHeader header "-"}}

Date: {{date}}
Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Contributor Author

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 :)

@irkartik
Copy link
Contributor Author

Hi @agjohnson just wanted to know the status of the PR. Is there anything to be updated from my side?

@agjohnson
Copy link
Contributor

@rajujha373 probably not, review just takes some time on our end.

@agjohnson agjohnson merged commit 4b28325 into readthedocs:master Mar 29, 2018
@irkartik
Copy link
Contributor Author

Thank you :) @agjohnson 🎉

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.

4 participants