Skip to content

Remove Date as an available type for msecs option of v1#249

Closed
rmm5t wants to merge 1 commit intouuidjs:masterfrom
rmm5t:248-fix-readme-v1-msecs-docs
Closed

Remove Date as an available type for msecs option of v1#249
rmm5t wants to merge 1 commit intouuidjs:masterfrom
rmm5t:248-fix-readme-v1-msecs-docs

Conversation

@rmm5t
Copy link
Copy Markdown

@rmm5t rmm5t commented Jan 3, 2018

Fixes #248

I figured this was the simplest approach and the most self-documenting considering that the option is named msecs.

@rmm5t rmm5t closed this Jan 3, 2018
@rmm5t rmm5t reopened this Jan 3, 2018
@rmm5t rmm5t mentioned this pull request Jan 6, 2018
@broofa
Copy link
Copy Markdown
Member

broofa commented Jan 6, 2018

Sorry, should have commented here earlier. The RunMD step needed some love so I've reworked this as #251

Thanks for the contribution, though!

@broofa broofa closed this Jan 6, 2018
@rmm5t rmm5t deleted the 248-fix-readme-v1-msecs-docs branch January 9, 2018 02:31
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.

2 participants