Skip to content

Bump Perl from 5.40 to 5.42. Remove some older versions from the CI test.#738

Merged
kotp merged 3 commits into
exercism:mainfrom
IsaacG:bump
Jun 8, 2026
Merged

Bump Perl from 5.40 to 5.42. Remove some older versions from the CI test.#738
kotp merged 3 commits into
exercism:mainfrom
IsaacG:bump

Conversation

@IsaacG

@IsaacG IsaacG commented Jun 3, 2026

Copy link
Copy Markdown
Member

@IsaacG

IsaacG commented Jun 6, 2026

Copy link
Copy Markdown
Member Author

@kotp Do you have the ability to rerun tests? And/or should some of these be dropped?

@kotp

kotp commented Jun 6, 2026

Copy link
Copy Markdown
Member

@kotp Do you have the ability to rerun tests?

I do.

And/or should some of these be dropped?

5.38 is EOL July 2nd of this year, and anything lower, we probably should not be testing for automatically.

So am for removing 5.38 via a logic switch, maybe. (A functional conditional no-op after a date query)

And of course, testing for EOL versions seems to be not all that valuable for what we are doing.

Comment thread .github/workflows/exercises.yml Outdated
Co-authored-by: Victor Goff <keeperotphones@gmail.com>
@IsaacG

IsaacG commented Jun 7, 2026

Copy link
Copy Markdown
Member Author

MacOS tests were fine for #734 but started timing out on #735

@kotp kotp left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Small change.

Also, we should look into the files not stating "use version" but use the version that is available instead,

Comment thread exercises/concept/booking-up-for-beauty/.meta/BookingUpForBeauty.pm Outdated
@kotp kotp self-requested a review June 7, 2026 23:48
@kotp

kotp commented Jun 7, 2026

Copy link
Copy Markdown
Member

@IsaacG I can bring this in regardless of the failing tests which currently target MacOS.

We can fix the CI in a different PR.

@IsaacG

IsaacG commented Jun 8, 2026

Copy link
Copy Markdown
Member Author

@IsaacG I can bring this in regardless of the failing tests which currently target MacOS.

We can fix the CI in a different PR.

Sounds good to me. I don't have a Mac so I'm not planning on debugging this MacOS issue. I don't have merge permissions for this PR.

@kotp kotp merged commit cd151e1 into exercism:main Jun 8, 2026
6 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants