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

CLN: Fix pep8 issues on test/test_find.py #1150

Merged
merged 18 commits into from
Mar 15, 2022

Conversation

LucyJimenez
Copy link
Contributor

xref #1042

@datapythonista datapythonista merged commit 3af84b8 into airspeed-velocity:master Mar 15, 2022
@datapythonista
Copy link
Member

Thanks @LucyJimenez

PR looks perfect. Your list of commits don't seem to make a lot of sense. A PR that you just opened shouldn't have 30 merge commits, not even one. You may want to review the video I shared with you on how to use git.

@LucyJimenez
Copy link
Contributor Author

Oh, thanks! I’m going to check the video and fix the problem :)

@LucyJimenez LucyJimenez deleted the pep8_p3 branch March 15, 2022 15:01
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.

None yet

2 participants