Refactor to improve code readability and re-use

[auxesis]

• Add a basic test suite, to ensure the scraper behaves consistently.
• Move all the call outs to GitHub and PlanningAlerts into their own class methods, to improve code readability and re-use.
• Update documentation to explain more clearly how to run it.
• Move environment variable configuration closer to the surface, to make it clearer how to run it.
• Load environment variables with dotenv in-process (instead of requiring the user to user the dotenv cli wrapper).
• Add validations to ensure the scraper bails early if the environment variables are not set.

[equivalentideas]

Thanks so much for this big contribution @auxesis This will make this project much more reusable ❤️

Would you mind resolving the merge conflict here and then assigning back to me to merge?

[equivalentideas]

@auxesis there's a bit of stuff here that could be simplified by using more standard rspec conventions. I'd be happy to merge this and refactor as we go, or to make those change before merging too. What would you prefer?

This is a great contribution @auxesis 💯

[auxesis]

@equivalentideas my preference would be to merge it, and clean up the tests in #5.

5:

• is a further refactor
• makes the code way easier to follow
• adds Right To Know stats

[equivalentideas]

my preference would be to merge it

Sounds good to me :shipit:

[equivalentideas]

@auxesis just ping me when that merge conflict is resolved and I'll merge it 👍

[auxesis]

@equivalentideas merge conflict fixed ✅

[equivalentideas]

:shipit: 🕶