Skip to content

feature: replace input queue with a structured string reader#498

Merged
Citymonstret merged 13 commits into2.0.0-devfrom
feature/2.0.0/commandinput
Nov 16, 2023
Merged

feature: replace input queue with a structured string reader#498
Citymonstret merged 13 commits into2.0.0-devfrom
feature/2.0.0/commandinput

Conversation

@Citymonstret
Copy link
Copy Markdown
Member

Some tests have been rewritten as they were super annoying to debug. I also re-wrote the flag parser as I had a lot of issues with it.

Some tests have been rewritten as they were super annoying to debug. I also re-wrote the flag parser as I had a lot of issues with it.
@Citymonstret Citymonstret added this to the 2.0.0 milestone Nov 13, 2023
@github-actions
Copy link
Copy Markdown

github-actions bot commented Nov 13, 2023

Test Results

  67 files  +  10    67 suites  +10   15s ⏱️ +2s
437 tests +196  437 ✔️ +196  0 💤 ±0  0 ±0 
440 runs  +198  440 ✔️ +198  0 💤 ±0  0 ±0 

Results for commit 31fd112. ± Comparison against base commit a76bdab.

♻️ This comment has been updated with latest results.

@Citymonstret Citymonstret self-assigned this Nov 14, 2023
@Citymonstret Citymonstret mentioned this pull request Nov 14, 2023
6 tasks
@Citymonstret Citymonstret linked an issue Nov 15, 2023 that may be closed by this pull request
Alexander Söderberg and others added 4 commits November 15, 2023 21:24
# Conflicts:
#	cloud-core/src/main/java/cloud/commandframework/CommandTree.java
# Conflicts:
#	cloud-annotations/src/main/java/cloud/commandframework/annotations/AnnotationParser.java
#	cloud-core/src/main/java/cloud/commandframework/arguments/CommandArgument.java
#	cloud-core/src/main/java/cloud/commandframework/arguments/preprocessor/RegexPreprocessor.java
@Citymonstret Citymonstret marked this pull request as ready for review November 16, 2023 22:33
@Citymonstret Citymonstret merged commit c13dcde into 2.0.0-dev Nov 16, 2023
@Citymonstret Citymonstret deleted the feature/2.0.0/commandinput branch November 16, 2023 22:33
@Citymonstret Citymonstret mentioned this pull request Nov 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

Default arguments aren't properly parsed

1 participant