search

exercism / elixir-analyzer

GNU Affero General Public License v3.0
30 stars 32 forks source link

Add div/rem related check to Sieve #206

Closed Br1ght0ne closed 2 years ago

Br1ght0ne commented 2 years ago

Close #193. Depends on https://github.com/exercism/website-copy/pull/2108.

jiegillet commented 2 years ago

You added tests for Kernel.//2 but you didn't add the assert_no_call did you?

jiegillet commented 2 years ago

Hi @Br1ght0ne, do you need help with this one?

jiegillet commented 2 years ago

@angelikatyborska the CI tests are gone, I don't know what I did. Have you seen this before?

angelikatyborska commented 2 years ago

I see this happen from time to time and there's never any explanation. I think this might be just GitHub's internal problems?

angelikatyborska commented 2 years ago

I just remembered - close and reopen a PR has the power to trigger Github CI