Skip to content

Conversation

@zhephyn
Copy link
Contributor

@zhephyn zhephyn commented Jul 3, 2025

Fixes issue #15 in the main foobara repository

@zhephyn
Copy link
Contributor Author

zhephyn commented Jul 3, 2025

I've renamed the method to remove the prefix and I've added the tests as well.

@zhephyn
Copy link
Contributor Author

zhephyn commented Jul 7, 2025

@azimux I've added the test which returns true. I've also run both rspec and rubocop locally and so far, no errors. Everything passes

@azimux
Copy link
Contributor

azimux commented Jul 7, 2025

w00t looks great! :shipit:

@azimux azimux merged commit 2afe420 into foobara:main Jul 7, 2025
2 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