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.

Sign up for GitHub

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

Prototype v2 #197

Open
wants to merge 19 commits into
base: master
Choose a base branch
from
Open

Prototype v2 #197

Changes from 1 commit
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
2e62846
drop support for multiple destination structs
alexflint Oct 4, 2022
09d28e1
split the parsing logic into ProcessEnvironment, ProcessCommandLine, …
alexflint Oct 4, 2022
22f214d
added test that library does not directly access environment variable…
alexflint Oct 4, 2022
a1e2b67
add a test to check that default values can be ignored if needed
alexflint Oct 4, 2022
4aea783
changed NewParser to take options at the end rather than config at th…
alexflint Oct 4, 2022
5ca19cd
cleaned up the test helpers parse, pparse, and parseWithEnv: now all …
alexflint Oct 4, 2022
5f0c48f
move construction logic out of parse.go into construct.go
alexflint Oct 4, 2022
2775f58
add OverwriteWithOptions, OverwriteWithCommandLine
alexflint Oct 4, 2022
64288c5
add appendToSlice, appendToMap, appendToSliceOrMap
alexflint Oct 4, 2022
b365ec0
add processSingle and make it responsible for checking whether an arg…
alexflint Oct 4, 2022
1cc263f
add processSequence and make it responsible for respecting "overwrite"
alexflint Oct 4, 2022
84b7154
add TestSliceWithEqualsSign
alexflint Oct 4, 2022
0769dd5
add tests for new Process* and OverwriteWith* functions
alexflint Oct 4, 2022
55d9025
rename "accumulatedArgs" -> "accessible"
alexflint Oct 4, 2022
60a0117
update readme for v2 (still has some TODOs)
alexflint Oct 7, 2022
47ff443
drop support for help tag inside arg tag
alexflint Oct 7, 2022
2ffe246
add mdtest command to generate and run tests from a markdown file
alexflint Oct 7, 2022
f2539d7
add go.work -- maybe remove before merge?
alexflint Oct 29, 2022
c046f49
drop go.work and add it to .gitignore
alexflint Oct 29, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions go.work
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
use .
use ./v2
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

go.work is meant to locally modify your workspace. It shouldn't be committed and shouldn't be part of a release. It's generally recommended to add it to .gitignore.

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!