Skip to content

Initialise project #1

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

Jump to bottom
Merged
zefir-git merged 30 commits into from
Sep 23, 2025
Merged

Initialise project #1

zefir-git merged 30 commits into from
Sep 23, 2025

Conversation

@zefir-git
Copy link
Contributor

@zefir-git zefir-git commented Sep 9, 2025 •
edited
Loading

TODO:

  • Finish the printHelp() method
  • Add CI
  • Examples in README
  • Documentation (outside of source)

@zefir-git zefir-git requested a review from kai-gitt September 9, 2025 17:51
@zefir-git zefir-git self-assigned this Sep 9, 2025
@zefir-git zefir-git added feature A new feature semver:minor New (backward compatible) functionality labels Sep 9, 2025
@zefir-git zefir-git marked this pull request as draft September 9, 2025 17:51
@zefir-git zefir-git marked this pull request as ready for review September 10, 2025 12:56
kai-gitt
kai-gitt requested changes Sep 20, 2025
View reviewed changes
zefir-git and others added 3 commits September 21, 2025 13:28
@zefir-git @kai-gitt
fix typo
1fabf9c 
Co-authored-by: かい <me@kai.enterprises>
@zefir-git @kai-gitt
fix ternary formatting (missing space)
a551811 
Co-authored-by: かい <me@kai.enterprises>
@zefir-git
move magic string to documented compile-time constant
6e07df1
@zefir-git
replace paddedName() in favour of a parameter in formattedName()
f3946d2
@zefir-git
fix order of args in doc for Option formattedName
f06f924
@zefir-git
fix Command.run() not delegating to sub-commands
9fddc70 
Bug was introduced in fb76e04.

The problem is that `parse()` delegates parsing to sub-commands, but
since it's now decoupled from `run()`, running was no longer delegated.

Accessing the delegated command during parsing via `ParsedArgs#command`.
@zefir-git zefir-git requested a review from kai-gitt September 21, 2025 12:08
@zefir-git zefir-git merged commit ad0fab5 into main Sep 23, 2025
1 check passed
@zefir-git zefir-git deleted the initialise branch September 23, 2025 08:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kai-gitt kai-gitt kai-gitt approved these changes

Assignees

@zefir-git zefir-git

Labels

feature A new feature semver:minor New (backward compatible) functionality

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@zefir-git @kai-gitt