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

Add ISBN Verifier Exercise #641

Merged
merged 2 commits into from
Feb 28, 2024

Conversation

tomasnorre
Copy link
Contributor

I just picked one by "random".

@mk-mxp mk-mxp added x:action/create Work on something from scratch x:knowledge/elementary Little Exercism knowledge required x:module/practice-exercise Work on Practice Exercises x:size/large Large amount of work x:rep/large Large amount of reputation labels Feb 26, 2024
Copy link
Contributor

@mk-mxp mk-mxp left a comment

Choose a reason for hiding this comment

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

Just one UUID missing for consistency. Thanks again for the valuable contribution!

@mk-mxp
Copy link
Contributor

mk-mxp commented Feb 26, 2024

@ErikSchierboom Do you have an idea, what really went wrong with the CI tests on MacOS? It looks like there is a library missing on the MacOS image. But that was not the case some hours ago. The image tag hasn't changed and the workflow is all the same. Any clues?

@ErikSchierboom
Copy link
Member

I don't, but what you could do is to pin to a specific version of the Mac runner in .github/workflows/exercise-tests-phpunit-9.yml. It's currently using latest which can cause these sudden breakages

@mk-mxp
Copy link
Contributor

mk-mxp commented Feb 26, 2024

@tomasnorre Please rebase to disable MacOS testing. This fails for reasons out of our control.

tomasnorre and others added 2 commits February 28, 2024 08:53
Co-authored-by: mk-mxp <55182845+mk-mxp@users.noreply.github.com>
@mk-mxp mk-mxp merged commit f475718 into exercism:main Feb 28, 2024
9 checks passed
@mk-mxp
Copy link
Contributor

mk-mxp commented Feb 28, 2024

Thanks for your patience with CI and me 😄

@tomasnorre
Copy link
Contributor Author

Thanks for your patience with CI and me 😄

No worry, we all want to do it as good as possible. Really appreciate your guidance.

@tomasnorre tomasnorre deleted the add-isbn-verifier-exercise branch February 28, 2024 11:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
x:action/create Work on something from scratch x:knowledge/elementary Little Exercism knowledge required x:module/practice-exercise Work on Practice Exercises x:rep/large Large amount of reputation x:size/large Large amount of work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants