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

Move timezone value under time #836

Merged
merged 1 commit into from
Jun 30, 2016
Merged

Move timezone value under time #836

merged 1 commit into from
Jun 30, 2016

Conversation

tas50
Copy link
Contributor

@tas50 tas50 commented Jun 30, 2016

This prevents us from breaking the various timezone cookbooks that
already use that namespace. Attributes are hard. Good thing lamont is
going to fix this all ;)

Signed-off-by: Tim Smith tsmith@chef.io

This prevents us from breaking the various timezone cookbooks that
already use that namespace. Attributes are hard. Good thing lamont is
going to fix this all ;)

Signed-off-by: Tim Smith <tsmith@chef.io>
@tas50
Copy link
Contributor Author

tas50 commented Jun 30, 2016

Ping @lamont-granquist @mcquin

@lamont-granquist
Copy link
Contributor

👍

@tas50
Copy link
Contributor Author

tas50 commented Jun 30, 2016

Tests are going to fail until we get a new chef gem out the door so we're going to have to merge this one with failing tests.

@tas50
Copy link
Contributor Author

tas50 commented Jun 30, 2016

Ping @btm

@btm
Copy link
Contributor

btm commented Jun 30, 2016

👍

@tas50 tas50 merged commit 4e619a8 into master Jun 30, 2016
@tas50 tas50 deleted the time branch June 30, 2016 20:44
@tas50 tas50 added the Bug label Jun 30, 2016
@thommay thommay added Type: Bug Does not work as expected. and removed Bug labels Jan 24, 2017
@chef chef locked and limited conversation to collaborators Nov 16, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Type: Bug Does not work as expected.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants