Changes in coriolis that require changes in alliance-test-suite are difficult

lip6 / coriolis

Coriolis VLSI EDA Tool (LIP6)

https://coriolis.lip6.fr

GNU General Public License v2.0

47 stars 5 forks source link

Changes in coriolis that require changes in alliance-test-suite are difficult #35

Open robtaylor opened 1 year ago

robtaylor commented 1 year ago

At the moment, if one needs to change alliance-check-toolkit for a change in coriolis, there's not a good sequence to do it in - you'll either have the new PR failing regression check, or main failing.

Any thoughts welcome!

Coloquinte commented 10 months ago

I grappled with the same issue. It would be nice to be able to run old commits with newer versions of alliance-check-toolkit as well. My solution would be to add a "ref" to the checkout of alliance-check-toolkit in .github/workflows/regressions.yml. This way we point a specific commit or tag that should work.

- uses: actions/checkout@v3
       with:
         repository: lip6/alliance-check-toolkit
         path:       coriolis-2.x/src/alliance-check-toolkit
         ref:        67b799e7b208