ansys / actions

A collection of reusable workflows
https://actions.docs.ansys.com/
MIT License
10 stars 5 forks source link

feat: add an optional input for checkout so it can be skipped, if needed #504

Closed myoung301 closed 3 months ago

myoung301 commented 3 months ago

The checkout remains the default, but can be skipped if set to false. This can be useful when there are prior steps to build elements that should be packed into the wheelhouse. For example, some projects have a step to build cython .pyd files and those .pyd files should go into the wheel file. The checkout step would purge the repo and clear the files from the previous build step. Now, projects that have this requirement can opt to skip the checkout.

ansys-reviewer-bot[bot] commented 3 months ago

Thanks for opening a Pull Request. If you want to perform a review write a comment saying:

@ansys-reviewer-bot review

myoung301 commented 3 months ago

@ansys-reviewer-bot review

ansys-reviewer-bot[bot] commented 3 months ago

Okay, I will trigger a review of your PR.

myoung301 commented 3 months ago

Fixed the syntax of the condition and verified it works with true, false, and without the parameter (defaults to true).

RobPasMue commented 3 months ago

LGTM! Merging. We will do a patch release with this change.

RobPasMue commented 3 months ago

Release on the way - v6 will have your change