Make deps.pex cache repo scoped, so PRs and main commits share deps.pex

dagster-io / dagster-cloud-action

Apache License 2.0

13 stars 20 forks source link

Make deps.pex cache repo scoped, so PRs and main commits share deps.pex #197

Closed shalabhc closed 2 months ago

shalabhc commented 2 months ago

Currently PRs can only reuse deps.pex if PRs do not change the dependencies. When dependencies are changed within a PR, the deps.pex is built every time. This is an optimization to let PRs build new deps.pex only once and then reuse it.

github-actions[bot] commented 2 months ago

Your pull request is automatically being deployed to Dagster Cloud.

Location	Status	Link	Updated
`from_gh_action`		View in Cloud	Sep 04, 2024 at 04:34 AM (UTC)

shalabhc commented 2 months ago

should we make this customizable in some way in the action? Do users have some way to manually clear the cache if they want to and they're using the github action, other than modifying the setup.py?

Currently there is no way to manually clear the cache but it's possible using the env var in the old github workflow or directly modifying the ci build command in the new workflow.