Skip to content

Implemented a Ruby version#6

Open
caitelatte wants to merge 3 commits intoEleanorRagone:masterfrom
caitelatte:master
Open

Implemented a Ruby version#6
caitelatte wants to merge 3 commits intoEleanorRagone:masterfrom
caitelatte:master

Conversation

@caitelatte
Copy link

Used advice on how to Ruby from RabidTech's Merube. This functions. :)

Possibly not functional yet. Heavily based on pccr's python
implementation.
Got advice from RabidTech's Rob and Mermop.
Understood the concept of self.<function> and fixed things. Works.
@EleanorRagone
Copy link
Owner

Can you just write some tests to verify that it works, and consistently works correctly? You can create a "ruby" folder, and move this file into that and write the test file there as well.

@caitelatte
Copy link
Author

Hey pcragone, thanks for the feedback. I'm working my way through the issues and improving it a bit slowly. :)

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