Skip to content
This repository has been archived by the owner on Aug 2, 2021. It is now read-only.

cmd/swarm: use only one function to parse flags #1536

Merged
merged 1 commit into from
Jul 4, 2019

Conversation

skylenet
Copy link
Contributor

@skylenet skylenet commented Jul 4, 2019

This is not required because calls like ctx.GlobalString(SwarmAccountFlag.Name) work for command line arguments and env vars.

@skylenet skylenet requested review from nonsense and janos July 4, 2019 07:50
@skylenet skylenet changed the title cmd/swarm: remove separate function to parse env vars cmd/swarm: use only one function to parse flags Jul 4, 2019
@skylenet skylenet requested a review from acud July 4, 2019 07:53
@skylenet skylenet added this to the 0.4.3 milestone Jul 4, 2019
@skylenet skylenet merged commit fb73e6c into master Jul 4, 2019
@skylenet skylenet deleted the remove-env-var-parsing branch July 8, 2019 08:18
vojtechsimetka added a commit that referenced this pull request Jul 9, 2019
* master:
  network/newstream: new stream! protocol base implementation (#1500)
  swarm: fix bzz_info.port when using dynamic port allocation (#1537)
  cmd/swarm: make bzzaccount flag optional and add bzzkeyhex flag (#1531)
  cmd/swarm: remove separate function to parse env vars (#1536)
  network/bitvector: Multibit set/unset + string rep (#1530)
  swarm: 0.4.3 unstable (#1526)
  travis: also build on release tags (#1527)
  swarm: release v0.4.2 (#1496)
  network: bump bzz stream hive (#1522)
  docker: update ca-certificates file (#1525)
  Add swarm guide to /docs (#1513)
  network/simulation: Add ExecAdapter capability to swarm simulations (#1503)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants