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.

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

Fix error in command.getResult() #105

Merged
merged 1 commit into from
Oct 11, 2023
Merged

Fix error in command.getResult() #105

merged 1 commit into from
Oct 11, 2023

Conversation

ebousse
Copy link
Contributor

@ebousse ebousse commented Oct 11, 2023

Fix a small bug in the code handling methods with a return value, where an index 0 was missing to retrieve the value in the resulting list.

@github-actions
Copy link

github-actions bot commented Oct 11, 2023

Test Results

110 tests  ±0   110 ✔️ ±0   1m 24s ⏱️ +3s
  16 suites ±0       0 💤 ±0 
  16 files   ±0       0 ±0 

Results for commit 50222d4. ± Comparison against base commit 5dc7241.

♻️ This comment has been updated with latest results.

@dvojtise dvojtise merged commit e2ec3b8 into master Oct 11, 2023
3 checks passed
@dvojtise
Copy link
Contributor

I've published the update site

do you need this change to also be published in maven central ? (this is more time-consuming since this requires to changes the version )

@ebousse
Copy link
Contributor Author

ebousse commented Oct 12, 2023

I personally don't use K3 using Maven, no on my side I do not need a publication on Maven Central. However having it in the next GEMOC Studio would be great, but I see you already took care of that :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants