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

Documentation for HTTP.rfc1123_date. #4418

Merged
merged 2 commits into from May 18, 2017

Conversation

akzhan
Copy link
Contributor

@akzhan akzhan commented May 16, 2017

@akzhan
Copy link
Contributor Author

akzhan commented May 16, 2017

@bew just note about akzhan@4c3890e#commitcomment-22167223: it's not true. A lot of documentation has same output without any puts.

Simply look for #.+# => regular expression in src folder.

But I'll remove assignment due to its useless. Update forced. Thanks.

@Sija
Copy link
Contributor

Sija commented May 16, 2017

string = part is IMO redundant since there's return type annotation.

@akzhan
Copy link
Contributor Author

akzhan commented May 16, 2017

@Sija,@bew thanks, assignment removed.

@Sija
Copy link
Contributor

Sija commented May 16, 2017

btw, thanks for filling gaps in documentation, much appreciated! 👍

@akzhan
Copy link
Contributor Author

akzhan commented May 17, 2017

@Sija Fixed.

P.S.: I will commit code without force updates due to keep comments history.

@Sija
Copy link
Contributor

Sija commented May 17, 2017

@akzhan no need to, they'll probably be squashed anyway.

@RX14
Copy link
Contributor

RX14 commented May 17, 2017

@Sija Keeping commit/comment history is useful for the duration of the PR, even if they will be squashed and not preserved when merged into master.

@bcardiff bcardiff added this to the Next milestone May 18, 2017
@bcardiff bcardiff merged commit d62caee into crystal-lang:master May 18, 2017
@bcardiff
Copy link
Member

Thanks again @akzhan ! As @Sija said, contributions to docs are important 👍

@akzhan akzhan deleted the doc-HTTP.rfc1123_date branch May 19, 2017 11:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants