Skip to content

Add planemo test workflow with bioconda dependency installation - #7

Merged
rvosa merged 8 commits into
mainfrom
copilot/add-bioconda-step-perl-bio-monophylizer
Dec 14, 2025
Merged

Add planemo test workflow with bioconda dependency installation#7
rvosa merged 8 commits into
mainfrom
copilot/add-bioconda-step-perl-bio-monophylizer

Conversation

Copilot AI commented Dec 14, 2025

Copy link
Copy Markdown
Contributor
  • Explore repository structure and understand existing CI/CD setup
  • Review existing planemo-lint.yml workflow
  • Review monophylizer.xml tool definition and tests
  • Create new GitHub Actions workflow for planemo test with bioconda
  • Configure workflow to install perl-bio-monophylizer from bioconda
  • Configure workflow to run planemo test monophylizer.xml
  • Validate workflow syntax
  • Refactor to use planemo's conda_auto_install feature
  • Improve shell configuration for error handling
  • Remove redundant Python version specification
  • Final code review
  • Security check with CodeQL - no vulnerabilities found
  • Remove defaults channel from conda configuration per feedback
  • Fix conda environment activation issue by using conda-managed Python and named environment
  • Add debugging step to show actual test output on failure for troubleshooting
Original prompt

Add to the CI/CD a step where perl-bio-monophylizer is installed from bioconda and then used to run planemo test monophylizer.xml as a unit test under the same conditions as the planemo linter


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI and others added 4 commits December 14, 2025 14:07
Co-authored-by: rvosa <106490+rvosa@users.noreply.github.com>
Co-authored-by: rvosa <106490+rvosa@users.noreply.github.com>
Co-authored-by: rvosa <106490+rvosa@users.noreply.github.com>
Co-authored-by: rvosa <106490+rvosa@users.noreply.github.com>
Copilot AI changed the title [WIP] Add CI/CD step for installing perl-bio-monophylizer Add planemo test workflow with bioconda dependency installation Dec 14, 2025
Copilot AI requested a review from rvosa December 14, 2025 14:14
@rvosa

rvosa commented Dec 14, 2025

Copy link
Copy Markdown
Member

@copilot don't use the conda defaults channel

Co-authored-by: rvosa <106490+rvosa@users.noreply.github.com>

Copilot AI commented Dec 14, 2025

Copy link
Copy Markdown
Contributor Author

@copilot don't use the conda defaults channel

Removed defaults channel from conda configuration in commit e3dc4bd. The workflow now only uses conda-forge and bioconda channels.

@rvosa

rvosa commented Dec 14, 2025

Copy link
Copy Markdown
Member

@copilot the planemo tests are failing. Something seems to be going wrong with activating the conda environment.

Co-authored-by: rvosa <106490+rvosa@users.noreply.github.com>

Copilot AI commented Dec 14, 2025

Copy link
Copy Markdown
Contributor Author

@copilot the planemo tests are failing. Something seems to be going wrong with activating the conda environment.

Fixed the conda environment activation issue in commit f5d9f5e. The problem was mixing setup-python and setup-miniconda actions. Now using conda-managed Python with a named environment ('planemo') that properly activates with the bash -el shell configuration.

@rvosa

rvosa commented Dec 14, 2025

Copy link
Copy Markdown
Member

@copilot the planemo test keeps failing and it says exactly 30 lines are wrong (which is the expected number of lines). Can we see the output that is being produced? I want to see if it could be a line breaks issue or sth.

Co-authored-by: rvosa <106490+rvosa@users.noreply.github.com>

Copilot AI commented Dec 14, 2025

Copy link
Copy Markdown
Contributor Author

@copilot the planemo test keeps failing and it says exactly 30 lines are wrong (which is the expected number of lines). Can we see the output that is being produced? I want to see if it could be a lin...

Added debugging output in commit b5983f7 that will display the actual test output on failure, including:

  • The generated output file content
  • Line count comparison
  • Hex dump of the last characters (to detect line ending issues)
  • Full diff between expected and actual output

This will help identify if it's a line breaks issue or other formatting problem. The debug output will appear in the CI logs when the test fails.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • wheels.galaxyproject.org
    • Triggering command: /home/REDACTED/.planemo/gx_venv_3/bin/pip pip install -r requirements.txt --index-url REDACTED --extra-index-url REDACTED (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

@rvosa
rvosa marked this pull request as ready for review December 14, 2025 17:51
@rvosa
rvosa merged commit fc363a7 into main Dec 14, 2025
3 of 6 checks passed
@rvosa
rvosa deleted the copilot/add-bioconda-step-perl-bio-monophylizer branch December 14, 2025 18:08
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