Wip/backup/print config message #11
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
feat: print derived config variables feat: print derived config variables in json test: 12 new unit tests to support features test: 1 skipped unit test for discovered bug in yargs with reports param
refactor: moving banner art to a function with a template literal refactoring test/print-config.js unit tests refactor: deprecating chai should statements refactor: require statments order refactor: moved before hook to inside the describe block refactor: changed skip syntax refactor: print-format unit tests to be more concise refactor test/help-message.js refactor: deprecating should statements refactor: requires in the header of the file docs: Adding jdocs styled comments docs: refactor todo notes fix: deprecating replaceAll for replace fix: updating snapshots test: unit test to cover gap in formatPrintVariable function refactor: cleanUpArgumentArray to make logic more concise refactor: addressing a gap in coverage test: adding a unit test for cleanUpArgumentArray docs: updating todo items and noting a gap in coverage refactor: config table generation is now split up into functions refactor,docs: fixed spelling mistakes refactor: restricting the --print-config-format flag to either text or json refactor: changing the name of the interface from cleanUpArgumentArray to cleanUpArgumentObject docs: adding jsdocs to new functions in lib/print-config.js fix: unit test with cross platform path concerns docs: fixing spelling mistakes test: unit test for printConfigText to cover gap docs: doc block for "Test: ensure config prints without a config file" docs: updating the readme to include print config flags refactor: unit test for no config file - compressed snapshot refactor: print config display to add padding refactor: print config to dynamically draw certain lines docs: adding doc blocks to unit test refactor: changing name of test helpers function file refactor: eslint disable lines refactor: eslint disable lines fix: fixing snapshop error by removing dynamic display components test: updating snapshot on windows docs: add todo items, misc notes feat: added longestColumn function test: longestColumn unit test docs: fixed some errors in jsDocs. Added jsDocs for some functions. fix: Error in text rendering of configuration due to padding bug. refactor: printing of objects and arrays. left a todo item. fix: test case failing on number of columns in config table test: added a test to printing objects fix: an error where the graphic and summary were not centered docs: adding comments to runspawn function refactor: removing lines from cleanJson function refactor: update to windows snapshots docs: updating notes in test helpers. Finished a todo and added a todo refactor: removed cleanJson function in test helpers refactor: cleaned up runSpawn funtion if logic refactor: new windowsPathProcessing function refactor: simplifying runSpawn function test: new snapshots docs: windowsPathProcessing jsdocs block feat: runc8 function wrapper for runSpawn refactor: updated runSpawn interface to support additional options docs: removed reg ex todos
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Checklist
npm test, tests passingnpm run test:snap(to update the snapshot)