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 reset timer #1838

Merged
merged 1 commit into from
May 16, 2016
Merged

Fix reset timer #1838

merged 1 commit into from
May 16, 2016

Conversation

matthewrmshin
Copy link
Contributor

It has lost the default=True setting due to #1743. Fix #1837.

@hjoliver @benfitzpatrick please review.

It has lost the default=True setting due to cylc#1743.
@matthewrmshin matthewrmshin added the bug Something is wrong :( label May 12, 2016
@matthewrmshin matthewrmshin added this to the next release milestone May 12, 2016
@hjoliver hjoliver closed this May 12, 2016
@hjoliver hjoliver reopened this May 12, 2016
@hjoliver
Copy link
Member

hjoliver commented May 12, 2016

(accidental PR closure rectified!)

@hjoliver hjoliver closed this May 12, 2016
@hjoliver hjoliver reopened this May 12, 2016
@hjoliver
Copy link
Member

hjoliver commented May 12, 2016

(again! the "close and comment" button seems to have moved, at least inside my head 😬)

@hjoliver
Copy link
Member

Review 1 - good; over to Ben.

@hjoliver hjoliver assigned benfitzpatrick and unassigned hjoliver May 14, 2016
@benfitzpatrick benfitzpatrick merged commit e66478b into cylc:master May 16, 2016
@matthewrmshin matthewrmshin deleted the fix-reset-timer branch May 16, 2016 09:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something is wrong :(
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants