/testing

checkers: ignore time zone in DeepEqual #115

Merged
merged 1 commit into from Oct 31, 2016

Conversation

Reviewers

@mhilton mhilton

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
4 participants
@rogpeppe @dimitern @jujubot @mhilton
@rogpeppe
Owner

rogpeppe commented Oct 31, 2016

This means tests won't need to concern themselves too deeply
about the time zone used in times unmarshaled from databases
or the network.

We also change the printed message to print the time nicely
rather than printing the details of the unexported time fields.
@rogpeppe
checkers: ignore time zone in DeepEqual 
This means tests won't need to concern themselves too deeply
about the time zone used in times unmarshaled from databases
or the network.

We also change the printed message to print the time nicely
rather than printing the details of the unexported time fields.
c081abc
@mhilton

mhilton approved these changes Oct 31, 2016
View changes

LGTM

@dimitern
Contributor

dimitern commented Oct 31, 2016

LGTM
@rogpeppe
Owner

rogpeppe commented Oct 31, 2016

$$merge$$
@jujubot
Contributor

jujubot commented Oct 31, 2016

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju-testing

@jujubot jujubot merged commit b868995 into juju:master Oct 31, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment