Modernizing this gem with a bit of love.

[kigster]

Note — this PR does not add or remove any functionality: it's a purely "fix this up" and bring this wonderful gem into 2018 by using correct formatting, newer rubies, and adding the missing dependency to the gemspec. I would love to be able to use this gem in my rewrite of the laser-cutter and may need to add some transformations such as rotations and shifting (unless they are already there), and I wanted to first submit a PR that, hopefully, should not be very controversial.

Note — also that the best way to see the real differences is to turn off white-space in git diff -w .... Unfortunately, due to the squashed commit, git was unable to recognize that the files were merely renamed, not changed.

And btw, the new gem I am writing is boxbot.

Changes in the PR

• adding mathn gem to the gemspec to make the tests pass locally and on CI
• renamed all test files to have _test.rb in them (minitest convention)
• used RubyMine + RubyCop to correctly reformat all white-space to use 2-spaces instead of tabs
• renamed README.markdown with README.md
• adding some misssing links to the README as well as the note about the dependency
• adding .idea to gitignore
• adding more recent ruby versions to Travis (all passing, after mathn is added)

[bfoz]

Boxbot looks like a good idea. I've done that sort of by hand way too many times. Enough so that I added a bit of DSL magic to my sketch gem, but it was never very pretty. I'm interested in seeing what you come up with.

Thanks for putting so much effort into this, but I'm going to reject it is as-is. Feel free to make changes and push it again if you're still interested.

• I'm a rather particular, and grumpy, old guss about whitespace and formatting. It is the way it is because that's the way I like it.
• Ditto for naming the test files
• The README extension I could see changing, but I'm not sure it's worthy of a commit
• I'd love to get rid of the mathn dependency, but I haven't had the time to look into it
• What's the .idea file for? I'm not familiar with it.
• I'd rather the README and Travis fixes be separate commits rather than a single mega-commit

[kigster]

No worries, I’ll use my own fork. Tabs are big no no in ruby, and naming test files correctly helps IDEs to find them.

Since you prefer heavily non standard configuration, you are on your own.

[kigster]

Note that your gem breaks on CI right now due lack of mathn dependency declared in the gemspec. Thought you might want to at the very least fix that.