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

Updating workflows/bacterial_genomics/bacterial_genome_annotation #538

Merged

Conversation

clsiguret
Copy link
Contributor

I fixed the test for several outputs identified in #530.
Thanks 😃

@clsiguret
Copy link
Contributor Author

clsiguret commented Sep 27, 2024

There is an error in the Comment PR:
Error: Unhandled error: HttpError: Validation Failed: {"resource":"IssueComment","code":"unprocessable","field":"data","message":"Body is too long (maximum is 65536 characters)"}

Is that why you said that "For bacterial_genome_annotation.ga_0 the log is too long" in #530, @lldelisle ?
How can I resolve this issue?

Thanks 😃

@lldelisle
Copy link
Contributor

Hi,
In fact, the tests are not passing and usually, the report for the test failure is commented here but it is too long to be in a comment. So you need to get the workflow test manually on the github action.
Click on 'Details' on the right of the "Galaxy Workflow Tests for push and PR / Combine...".
Click on the top left on 'Summary'.
On the bottom of the page, click on All tool test results this will allow you to download a zip with inside a html with the workflow test result.
Then, you should be able to see what went wrong among your tests.

@lldelisle lldelisle merged commit 8358582 into galaxyproject:main Sep 27, 2024
6 checks passed
@lldelisle
Copy link
Contributor

Thank you very mych @clsiguret

@clsiguret clsiguret deleted the bacterial-genome-annotation_update branch September 27, 2024 16:18
@mvdbeek
Copy link
Member

mvdbeek commented Sep 27, 2024

Attention: deployment skipped!

https://github.com/galaxyproject/iwc/actions/runs/11073779185

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.

3 participants