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

Fix #unix_ms to #to_unix_ms in CANGELOG.md #7024

Merged
merged 3 commits into from
Nov 7, 2018
Merged

Fix #unix_ms to #to_unix_ms in CANGELOG.md #7024

merged 3 commits into from
Nov 7, 2018

Conversation

arcage
Copy link
Contributor

@arcage arcage commented Nov 5, 2018

The CHANGELOG of the 0.27.0 shows that some instance method names are changed in Time class.

  • (breaking-change) Rename Time#epoch to Time#to_unix. Also #epoch_ms to #unix_ms, and #epoch_f to #to_unix_f. (#6662, thanks @straight-shoota)

Actually, #epoch_ms was renamed to #to_unix_ms, not #unix_ms.

The CHANGELOG for 0.27.0 shows that some instance method names are changed in Time class.

> - **(breaking-change)** Rename `Time#epoch` to `Time#to_unix`. Also `#epoch_ms` to `#to_unix_ms`, and `#epoch_f` to `#to_unix_f`. ([#6662](#6662), thanks @straight-shoota)

Actually, `#epoch_ms` was renamed to `#to_unix_ms`, not `#unix_ms`.
Copy link
Member

@sdogruyol sdogruyol left a comment

Choose a reason for hiding this comment

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

Thank you @arcage 👍

@j8r
Copy link
Contributor

j8r commented Nov 5, 2018

@arcage little typo, H missing for CHANGELOG in the commit title.

@arcage
Copy link
Contributor Author

arcage commented Nov 5, 2018

@j8r Oops, sorry.

I've tried to correct the commit title.

Is this OK?

Or, should I close this PR and reopen with correct commit title?

@straight-shoota
Copy link
Member

This is fine 👍

@j8r
Copy link
Contributor

j8r commented Nov 6, 2018

It'll be rebased, the commit title & message can be edited.

@RX14 RX14 added this to the 0.27.1 milestone Nov 7, 2018
@RX14 RX14 added the kind:bug A bug in the code. Does not apply to documentation, specs, etc. label Nov 7, 2018
@RX14 RX14 merged commit b428782 into crystal-lang:master Nov 7, 2018
@bcardiff bcardiff added kind:docs and removed kind:bug A bug in the code. Does not apply to documentation, specs, etc. labels Jan 26, 2019
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.

6 participants