Codecov Report

❗ No coverage uploaded for pull request base (v3-development-master@c5f50db). Click here to learn what that means.
The diff coverage is 64.18%.

@@                   Coverage Diff                    @@
##             v3-development-master    #1058   +/-   ##
========================================================
  Coverage                         ?   67.77%           
========================================================
  Files                            ?       31           
  Lines                            ?     2647           
  Branches                         ?        0           
========================================================
  Hits                             ?     1794           
  Misses                           ?      741           
  Partials                         ?      112

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c5f50db...120695c. Read the comment docs.

Motivation, for context: #907 (comment)

This might be something to balance against the goal of making the project more modular, re: #1055. Two of our three dependencies, github.com/BurntSushi/toml and gopkg.in/yaml.v2, come from the altsrc package.

This PR doesn't intend to make the project modular. It simply exists to integrate the altsrc into the main package. To make the library modular is the last thing in my plan when I am sure that the core features intended for v3 are already in the development branch.

Once done, I can think of functionality that can be offloaded into a separate module and downloaded on demand by a user who wants it.

Sorry, I may have phrased that poorly.

This change seems to move in the opposite direction of modularity. We're taking functionality that comes with dependencies such as yaml and toml-parsing and moving that from a separate package into the cli package. Not that there isn't reason to do this—modularity obviously isn't our only goal. If modularity is one of our goals, though, I wanted to flag that we're bringing more dependencies into our core package by making this change, which might make it harder to stay modular in the future, unless we have a specific path forward on that front.

I agree with you @rliebz. Apologies if I came too strongly there. What I wanted to say is I just wanted all the proposed changes together in the development branch to see how they work out together. Once the features are frozen, I would proceed with a cleanup and extract such optional components into separate modules which can then be imported by users when they want to use them.

I @asahasrabuddhe @rliebz I think you both just made a very strong argument for us including a "module system" in the v3 release!

I agree with @asahasrabuddhe's proposal of merging altsrc into cli, and then moving it out again later - after the module system is in.

This issue or PR has been automatically marked as stale because it has not had recent activity. Please add a comment bumping this if you're still interested in it's resolution! Thanks for your help, please let us know if you need anything else.

Closing this as it has become stale.