Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change global.json rollForward to latestPatch once GitHub Actions supports it #171

Open
MattKotsenas opened this issue Aug 1, 2024 · 0 comments

Comments

@MattKotsenas
Copy link
Collaborator

MattKotsenas commented Aug 1, 2024

Tracking issue for #170.

Once GitHub Actions supports the required rollForward strategies (tracked at actions/setup-dotnet#448), we should switch from our current disable to latestPatch.

Alternatively, we can use renovate to auto-update global.json, but that only addresses CI and not local development.

MattKotsenas added a commit that referenced this issue Aug 1, 2024
Update .NET SDK to the latest released version and disable
`rollForward`.

Updates to the .NET SDK feature band can introduce new / updated
analyzers (for instance, 8.0.400 adds
[IDE0320](dotnet/roslyn#73012)). As a result,
updating the feature band isn't "safe" to do from a build
reproducibility perspective. It will also conflict with #166.

That means the roll forward options available to us are:

- `patch`
- `latestPatch`
- `disable`

However, GitHub Actions don't support these options (tracked by
actions/setup-dotnet#448). Thus, the only
`rollForward` strategy that currently does the same thing locally and in
CI is `disabled`.

Once 448 is fixed, we can / should probably switch to `latestPatch`
(tracked by #171).

This 'gotcha' is also added to the SquiggleCop documentation
[here](https://github.com/MattKotsenas/SquiggleCop?tab=readme-ov-file#common-sources-of-baseline-mismatches).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants