Skip to content
This repository has been archived by the owner on Jul 10, 2024. It is now read-only.

✍️ Added ability to parse arguments #26

Draft
wants to merge 8 commits into
base: main
Choose a base branch
from
Draft

✍️ Added ability to parse arguments #26

wants to merge 8 commits into from

Conversation

Bullrich
Copy link
Contributor

@Bullrich Bullrich commented May 22, 2024

Resolves #8

  • Added a method which handles arguments and verifies that it fulfills its condition
  • Added JOI validation to handle the types:
    • one_of
    • string
    • regex
  • Created a custom error to handle nested problems (easier to read)
  • Added a parameter section to the command interface

Todo

  • Update the JSON schema
  • Add more test with failing cases
  • Update the generated documentation

@Bullrich Bullrich added this to the GitHub Action milestone May 22, 2024
@Bullrich Bullrich self-assigned this May 22, 2024
@Bullrich Bullrich linked an issue May 22, 2024 that may be closed by this pull request
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Handle arguments in commands
1 participant