Skip to content

Instantly share code, notes, and snippets.

@JStans12
Last active February 22, 2017 18:28
Show Gist options
  • Save JStans12/5695d40174feb4c2f36b4837e26b168c to your computer and use it in GitHub Desktop.
Save JStans12/5695d40174feb4c2f36b4837e26b168c to your computer and use it in GitHub Desktop.

Contributing to Figaro

  • Why: I use Figaro on every rails project to hide environment variables. If I can help make it better, I should.

* What: There is a bug where running `bundle exec figaro install` will append an ignore line in the .gitignore even if that line already exists. Seems reasonable. laserlemon/figaro#259
* I need: To dig into Figaro to figure out where that line is being added and see what changes need to be made. The first step will be to reproduce the bug.
* When I'm Done: There's not much conversation around the topic, but if the bug is fixed, I don't see why it wouldn't be merged.

Contributing to Shoulda-Matchers

  • Why: I use it all the time. I've had a few problems with it, but can't remember exactly what they were right now.

* What: There is an issue where `.not_to validate_length_of` produces a misleading message in rspec which says "Expected User not to validate that the length of :password, but it did." This message needs to make more sense. thoughtbot/shoulda-matchers#994
* I need: To figure out what's going on and reword the rspec message. First step is to reproduce.
* When I'm done: There is support for the feature from thoughtbot so hopefully it would get merged.

Contributing to Faker

  • Why: Another thing that I use all the time that could be improved.

* What: Currently, you can generate different parts of an address, but not a full address. This would be a simple PR to create. faker-ruby/faker#696
* I need: To write a method to fake a full address. Will mostly be string interpolation.
* When I'm done: I'll create a PR and hope it is merged. The issue has been open for a while without support from community, but not main team.
@jcasimir
Copy link

Figaro seems very low-hanging and straightforward. I like it.

Shoulda-Matchers is also good. RSpec internals can be tricky.

Faker one is good but the other two seem a little more interesting.

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