Skip to content

Add isbn-verifier - #318

Merged
kotp merged 1 commit into
exercism:mainfrom
BNAndras:add-isbn-verifier
Aug 25, 2026
Merged

Add isbn-verifier#318
kotp merged 1 commit into
exercism:mainfrom
BNAndras:add-isbn-verifier

Conversation

@BNAndras

@BNAndras BNAndras commented Aug 5, 2026

Copy link
Copy Markdown
Member

No description provided.

@BNAndras BNAndras added x:module/practice-exercise Work on Practice Exercises x:rep/large Large amount of reputation labels Aug 5, 2026
@BNAndras

BNAndras commented Aug 5, 2026

Copy link
Copy Markdown
Member Author

CI is breaking due to bank-account tests being a bit flaky. I'll take a closer look when I have some free time later this week or next week.

@kotp

kotp commented Aug 5, 2026

Copy link
Copy Markdown
Member

CI is breaking due to bank-account tests being a bit flaky. I'll take a closer look when I have some free time later this week or next week.

It is approved, but I will leave it up to "not me" if this comes in before or after the fix for the tests in general.

@BNAndras

BNAndras commented Aug 5, 2026

Copy link
Copy Markdown
Member Author

Nah, we should fix bank account first and then rebase this PR before merging. That's a bit more priority in my book.

@kotp

kotp commented Aug 5, 2026

Copy link
Copy Markdown
Member

Nah, we should fix bank account first and then rebase this PR before merging. That's a bit more priority in my book.

That is also my feeling on the subject.

@kotp
kotp merged commit a07b575 into exercism:main Aug 25, 2026
3 checks passed
@BNAndras
BNAndras deleted the add-isbn-verifier branch August 25, 2026 14:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

x:module/practice-exercise Work on Practice Exercises x:rep/large Large amount of reputation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants