Pull latest Ruff image on upgrade-test by jeremybeard · Pull Request #1843 · astronomer/astro-cli

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this method feels slightly wrong to me right now. Mostly I had written it or reviewed it sometime in the past 🙈 . But thinking about it now, why do we even have two separate ways to pass image name?

This is not a blocker at all, just throwing a comment to stir a conversation on the right approach.

I think the right approach here might be to drop the remoteImage arg and force everyone to use the imageName from DockerImage, since the intention of the interface was to club the operation around an image.