Skip to content

[bugfix] Fixes #3520, parseZone incorrectly handled minutes under 16 - #3643

Closed
maggiepint wants to merge 1 commit into
developfrom
maggie/offsetminutesashours
Closed

maggiepint wants to merge 1 commit into
developfrom
maggie/offsetminutesashours

Conversation

@maggiepint

Copy link
Copy Markdown
Member

Fixes issue in #3520 where explicitly parsed offset in minutes less than abs(16) is converted to hours.

@icambron

icambron commented Dec 5, 2016

Copy link
Copy Markdown
Contributor

LGTM

@maggiepint

Copy link
Copy Markdown
Member Author

Note: the .vscode in the .gitignore allows moment to be used with Visual Studio Code's node debugger. Given how popular the editor has become, I can't see that it hurts.

@icambron

icambron commented Dec 5, 2016

Copy link
Copy Markdown
Contributor

@maggiepint In theory you should block that in your global .gitignore file. In practice I agree it doesn't hurt.

@ichernev

ichernev commented Feb 4, 2017

Copy link
Copy Markdown
Contributor

LGTM

@ichernev

ichernev commented Mar 2, 2017

Copy link
Copy Markdown
Contributor

Merged in 5c54613

@ichernev ichernev changed the title fix conversion of parsed offset to hours [bugfix] Fixes #3520, parseZone incorrectly handled minutes under 16 Mar 2, 2017
@ichernev ichernev closed this Mar 2, 2017
ichernev added a commit that referenced this pull request Mar 2, 2017
[bugfix] Fixes #3520, parseZone incorrectly handled minutes under 16
@mattjohnsonpint
mattjohnsonpint deleted the maggie/offsetminutesashours branch August 13, 2026 18:42
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.

4 participants