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

docker: skip running node if offline var is set #278

Merged
merged 1 commit into from
Feb 14, 2022

Conversation

pro-wh
Copy link
Collaborator

@pro-wh pro-wh commented Feb 11, 2022

this is only an optimization. we recommend that users don't rely on software configuration to operate offline.

@tjanez
Copy link
Member

tjanez commented Feb 11, 2022

Hmm... why was CI not run for this PR?

@pro-wh
Copy link
Collaborator Author

pro-wh commented Feb 11, 2022

turns out this doesn't work well with the set -u. gonna adjust it

@pro-wh
Copy link
Collaborator Author

pro-wh commented Feb 11, 2022

maybe the ci thing has to do with the base branch

docker/service/node/run Outdated Show resolved Hide resolved
@tjanez
Copy link
Member

tjanez commented Feb 11, 2022

maybe the ci thing has to do with the base branch

Oh, yeah, that is the reason

@tjanez
Copy link
Member

tjanez commented Feb 11, 2022

@pro-wh, do we want to include this one in the 2.0.0 release?

@pro-wh
Copy link
Collaborator Author

pro-wh commented Feb 11, 2022

@pro-wh, do we want to include this one in the 2.0.0 release?

sure, if you wouldn't mind rearranging

@tjanez tjanez added this to the 2.0.0 milestone Feb 14, 2022
@pro-wh pro-wh changed the base branch from changelog to master February 14, 2022 19:37
Co-authored-by: Tadej Janež <tadej.j@nez.si>
@pro-wh
Copy link
Collaborator Author

pro-wh commented Feb 14, 2022

rebasing, adding changelog

@pro-wh pro-wh merged commit dd388a6 into master Feb 14, 2022
@pro-wh pro-wh deleted the pro-wh/feature/offline branch February 14, 2022 19:56
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.

None yet

3 participants