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

Provide Docker information in README #1769

Merged
merged 1 commit into from Mar 5, 2019

Conversation

Projects
None yet
3 participants
@alberto56
Copy link
Contributor

alberto56 commented Mar 5, 2019

No description provided.

@jwiegley jwiegley merged commit 6fa1c46 into ledger:next Mar 5, 2019

1 check failed

continuous-integration/travis-ci/pr The Travis CI build could not complete due to an error
Details
@crazyh

This comment has been minimized.

Copy link

crazyh commented Mar 14, 2019

Not sure that this image is usable for all: it contains hard coded timezone.

@alberto56

This comment has been minimized.

Copy link
Contributor Author

alberto56 commented Mar 14, 2019

Hi @crazyh -- I'd be interested in obtaining more information about how ledger uses timezone information; I have also opened dcycle/docker-ledger#2 to track this within the Docker project itself.

@crazyh

This comment has been minimized.

Copy link

crazyh commented Mar 14, 2019

I do not know how ledger uses timezone for calculations but it uses strftime so it is probably that some operations like pricedb-related should be affected by TZ.

@alberto56

This comment has been minimized.

Copy link
Contributor Author

alberto56 commented Mar 21, 2019

Have just merged in a pull request which no longer uses a hard-coded timezone, see dcycle/docker-ledger#2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.