Skip to content

change week display rule#21

Merged
dnephin merged 1 commit intodocker:masterfrom
allencloud:change-week-display-rule
Oct 20, 2016
Merged

change week display rule#21
dnephin merged 1 commit intodocker:masterfrom
allencloud:change-week-display-rule

Conversation

@allencloud
Copy link
Contributor

Since the original code will NEVER show 2 months, this pr fixes that.

Signed-off-by: allencloud allen.sun@daocloud.io

@allencloud allencloud force-pushed the change-week-display-rule branch from d8bb27f to 2b437e2 Compare October 20, 2016 09:08
Signed-off-by: allencloud <allen.sun@daocloud.io>
@allencloud allencloud force-pushed the change-week-display-rule branch from 2b437e2 to 61a1eab Compare October 20, 2016 09:08
@allencloud
Copy link
Contributor Author

ping @thaJeztah @vdemeester

Copy link
Collaborator

@vdemeester vdemeester left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🐸

Copy link
Contributor

@dnephin dnephin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@dnephin dnephin merged commit 8a7beac into docker:master Oct 20, 2016
thaJeztah added a commit to thaJeztah/containerd that referenced this pull request Apr 23, 2019
relevant changes:

- docker/go-units#19 make 1 second not to be plural seconds
- docker/go-units#20 Add `HumanSizeWithPrecision` function
- docker/go-units#21 change week display rule
- docker/go-units#22 Better human duration precision
- docker/go-units#23 Removes spaces before unit
- docker/go-units#27 Fix containerd#26 - RAMInBytes Bug
- docker/go-units#33 Fix handling of unlimited (-1) ulimit values
- docker/go-units#34 Revert 46 minute threshold

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
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.

3 participants