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

Install system dependencies in the workflows #198

Closed
Gotfrid opened this issue Jan 7, 2025 · 2 comments
Closed

Install system dependencies in the workflows #198

Gotfrid opened this issue Jan 7, 2025 · 2 comments
Assignees

Comments

@Gotfrid
Copy link

Gotfrid commented Jan 7, 2025

Hi, in aNCA project we need to take a dependency on the units package - which requires a corresponding system dependency (libudunits2-dev).

This project uses admiralci workflows, however most of these workflows have run-system-dependencies: false, which makes it impossible for us to run even spellcheck.

I'm opening this issue seeking for advice on how to best solve this problem. We can either inline all the workflows that we currently use and toggle system dependency installation, or we can change admiralci workflows.

The former might be a short-term remedy, but is very undesirable as we would instantly fall behind if any updates are to happen here.

Would it make sense to add something like install-system-dependencies to the workflow inputs with a default false value, so that it doesn't affect existing projects? Ofc I'd be willing to open a PR for that.

cc @m-kolomanski

@cicdguy
Copy link
Collaborator

cicdguy commented Feb 18, 2025

Looks like you copied the core workflow verbatim in pharmaverse/aNCA#151 and modified it there. No action needed here.

@cicdguy cicdguy closed this as completed Feb 18, 2025
@Gotfrid
Copy link
Author

Gotfrid commented Feb 20, 2025

Indeed, we copied the workflow as a temporary suboptimal solution.

To solve this problem properly, my suggestion is to parametrize run-system-dependencies and make it an input in every workflow that uses insightsengineering/staged-dependencies-action. This way, you can keep it false by default to ensure backwards compatibility with the projects that already use this repository to run actions, but also give some flexibility to some projects that need it.

In my opinion, unit conversion is not an uncommon thing in biological, chemical, pharmaceutical projects, which often requires a dependency on a system library libudunits2-dev. Currently, a fellow pharmaverse project aNCA needs it, hence this issue.

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

No branches or pull requests

2 participants