svalinn / DAGMC

Direct Accelerated Geometry Monte Carlo Toolkit
https://svalinn.github.io/DAGMC
Other
96 stars 61 forks source link

CI requires contributors to build/publish packages #914

Closed gonuke closed 9 months ago

gonuke commented 9 months ago

Describe the Bug
In recent changes to the CI system, we rely on pre-built packages with dependencies. The current scheme requires each contributor to have the full set of packages on their own fork, which may limit contributions.

To Reproduce
Create a pull request from a fork that has not already build the required packages.

Expected Behavior
Contributors should not have to already build the dependency packages on their own forks.

Additional Context
The current scheme is convenient for contributors who want to test changes that require or depend on changes to the dependencies because it does seek out packages on their own fork. If a contributor would like to test a change that requires modified dependencies, they will need to develop their own workflow for that.

This issue was noticed nearly simultaneously in cyclus by @bennibbelink and in DAGMC by @ahnaf-tahmid-chowdhury