cake-build / cake-action

:cake::octocat: Run Cake (C# Make) scripts in your GitHub Actions workflows.
https://cakebuild.net
MIT License
45 stars 13 forks source link

The latest Cake version is not always honoured #55

Closed ecampidoglio closed 2 months ago

ecampidoglio commented 6 months ago

Background

Running the action without specifying the --cake-version input parameter implies installing the latest version of the Cake.Tool package. This is achieved by running dotnet tool install without the --version parameter.

The Problem

While this approach works fine in most cases, there's one scenario where it falls short: when the action runs on an agent where a different version of the Cake.Tool package is already installed.

Here's an example workflow:

  1. The Cake action runs with the --cake-version set to x.y.z
  2. Version x.y.z of the Cake.Tool is installed in the tools directory
  3. The Cake action runs again, only this time without the --cake-version input parameter implying that the latest version of Cake should be used
  4. The action sees that the Cake.Tool is already present in the tools directory; however, without a specific version number to compare with, it determines that it shouldn't install any new version of the Cake.Tool and runs with it
  5. The user thinks that the action is running with the latest version of Cake, while in reality it's running with whatever version was alreaady installed in the tools directory

Proposed Solution

When running the action without the --cake-version input parameter, the action should go out and fetch the latest version of the Cake.Tool from the Cake releases on GitHub and use that as the version number.

This way, there will always be a version number to compare with when determining whether any existing installation of the Cake.Tool can be used.

Risks

In the case when fetching the latest version from GitHub fails for some reason (e.g. due to a network error), the action should simply fall back to the previous behavior of installing the Cake.Tool without a version number.

baynezy commented 2 months ago

@ecampidoglio

In the case when fetching the latest version from GitHub fails for some reason (e.g. due to a network error), the action should simply fall back to the previous behavior of installing the Cake.Tool without a version number.

I am not sure I agree with it silently falling back to the installed verrsion. If you consider the current behaviour a bug, then I don't think you can make the fallback behaviour of this change the same behaviour.

ecampidoglio commented 2 months ago

@baynezy I agree with you in principle. However, there are a few things to consider here:

  1. The original behavior is only problematic in a specific fairly uncommon scenario
  2. The vast majority of GitHub Actions workflows run on GitHub's own servers, so not being able to retrieve the latest Cake release would be a pretty rare occurrence

Given these two premises, I think that, from a practical standpoint, falling back on the original behavior is a better alternative than completely failing the build.