New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Minor documentation fix (cartesin -> cartesian) #4

merged 1 commit into from Jan 26, 2015


None yet
2 participants

singularperturbation commented Jan 22, 2015


I feel sheepish submitting a small (documentation) PR like this, but this is my reasoning:

I was originally going to try and work on this for the CPAN PR challenge/CPAN Lottery, but I ran into a few problems:

  1. My Perl is pretty rusty.
  2. Your code quality is good, is self-contained, and has unit tests. (So no low-hanging fruit). :)
  3. My knowledge of set operations/set theory is pretty basic.

Anyway, you have a very nice CPAN module, and I didn't want to let the month go by without doing anything. I thought that this would be at least a little helpful- let me know if there's another area (different unit tests, another function/set operation), etc. that would be more helpful and I'd be glad to dig in/help if I can.

aarondcohen added a commit that referenced this pull request Jan 26, 2015

Merge pull request #4 from singularperturbation/master
Minor documentation fix (cartesin -> cartesian)

@aarondcohen aarondcohen merged commit e313164 into aarondcohen:master Jan 26, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment