Skip to content
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

has_columns decorator #38

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

shivam6294
Copy link

@shivam6294 shivam6294 commented May 29, 2017

Hey man! Thanks for the amazing repo! It's an elegant pythonesque way for validating data in a Pandas DataFrame.has_columns is another check that is performed quite often in my workflow, so I thought of adding this to a common place, aka your repo. In this PR, I have added a decorator has_columns, for checking if a DataFrame has all the columns specified in a list. Fixed PEP8 errors where possible and added Pycharm xml files to .gitignore.

@TomAugspurger
Copy link
Collaborator

Hi, this looks great. I'll take a closer look next week and get it merged (and cut a release). Sorry for the delay.

@lucianoviola
Copy link

Any updates?

@TomAugspurger
Copy link
Collaborator

TomAugspurger commented May 26, 2018 via email

@alysivji
Copy link
Contributor

@TomAugspurger Lemme know if you need a hand with engarde. I've used it in a side project, will be bringing it into work projects over the next few months.

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.

4 participants