Skip to content

fix sys gnuplot test#205

Merged
t-bltg merged 1 commit into
mbaz:masterfrom
t-bltg:sys-gnuplot
May 10, 2026
Merged

fix sys gnuplot test#205
t-bltg merged 1 commit into
mbaz:masterfrom
t-bltg:sys-gnuplot

Conversation

@t-bltg

@t-bltg t-bltg commented May 10, 2026

Copy link
Copy Markdown
Collaborator

@mbaz I do'nt understand the purpose of de67a8f.

When using @latest suffix, this ensure that we use the latest setup-julia actions, hence we don't have to manually change numbers e.g. @v3 to @v4.

Fix for de67a8f, making the preferences test against the system gnuplot conditional.

@t-bltg t-bltg added the CI label May 10, 2026
@t-bltg
t-bltg merged commit eb9ac2d into mbaz:master May 10, 2026
6 of 7 checks passed
@t-bltg
t-bltg deleted the sys-gnuplot branch May 10, 2026 12:27
@mbaz

mbaz commented May 10, 2026

Copy link
Copy Markdown
Owner

@t-bltg Did you see https://discourse.julialang.org/t/deprecating-the-latest-and-vx-y-tags-of-the-julia-actions-setup-julia-github-action/136936 ? Am I misunderstanding that the latest tag is deprecated?

@t-bltg

t-bltg commented May 10, 2026

Copy link
Copy Markdown
Collaborator Author

No, I haven't, thanks for pointing that out.

Using PRs with a description (and the discourse url in it), linking to the commit would have helped understanding why @latest was removed.

@mbaz

mbaz commented May 10, 2026

Copy link
Copy Markdown
Owner

I did link the conversation in the commit message. I'm used to just committing locally and pushing, but you make a good point that using PRs would be clearer.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants