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

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

Closed
wants to merge 1 commit into from
Closed

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

wants to merge 1 commit into from

Conversation

rmm5t
Copy link

@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
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