feat: show derived configuration

[mcknasty]

Pull Request 517

Commit Message

commit b33e8e7773ea79897c8b77f92894a7cb3fc5aaf7 Author: Trevor D McKeown [email protected] Date: Sun Feb 18 09:17:58 2024 -0500

Print Derived Configuration Report (#517)

feat: print derived config variables
feat: print derived config variables in json
test: 16 new unit tests to support features
test: 1 skipped unit test for discovered bug in yargs with reports param

Following the Contributions Recommendations here.

1. [x] Make sure you have installed the latest version of Node.js
2. [x] Fork this project on GitHub and clone your fork locally
3. [x] Create local branches to work within. These should also be created directly from the main branch. Local Fork here.
4. [x] Make your changes.
5. [x] Run tests to make sure all is okay (everything should pass except the snapshot).
   1. A complete log of initial test results.
   2. As instructed, ignore snapshot failures. 1 pending
   3. 111 passing in 54 seconds
6. [x] Now update the snapshot.
   1. A complete log of snapshot test results.
   2. 111 passing in 54 seconds
7. [x] If all is passing, commit your changes. The log of the commit can be found here.
8. [x] As a best practice, once you have committed your changes, it is a good idea to use git rebase (not git merge) to synchronize your work with the main repository.
9. [x] Run tests again to make sure all is okay.
   1. A complete log of the final test results.
   2. 111 passing in 55 seconds
10. [x] Push
11. [x] Open the pull request, see details in the template.
12. [ ] Make any necessary changes after review.

[mcknasty]

Saw a few nit picks. I might be able to refactor this patch to use fewer files, by naming the snapshots in the test cases. WIP.

[mcknasty]

Thanks @bcoe. I will make the changes above and have a new commit in the next week.

[mcknasty]

@bcoe, Please review again.

[bcoe]

@mcknasty it's mostly my fault for not being more engaged on this repo, but I think I need to declare bankruptcy on some of the inflight PRs.

If you're still feeling passionate about some of these changes (some were pretty cool IMO, like printing derived configuration), mind reopening on at a time, with a tracking issue that describes the functionality you'd like to see.

Thank you for the work you have done on this repo 👏

[mcknasty]

Thanks for your comments on these. Some of the PRs did need to get closed. Unfortunately, this one is a prerequisite for #518 at the moment.