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
OO API + black formatting for episode 5 #88
Comments
To me it looks like episode 5 is already converted though, right? Or did I miss something? |
when I ran black command, I see the following output: 1 file reformatted for some of the snippets This is only for 4 of the snippets or so |
Yes, you are right. Episode 5 is already converted. Any other episode that needs to be converted? |
Oh, if there's more, feel free to open PRs for it. I'm not aware of any other snippets other than those of episode 1 (#83) |
Hi, I tried to run black on some snippets and the comparison can be seen here: Could you please take a look and see if this is what we want? (I am a bit confused here). Thanks. |
Oh, indeed!! I had only taken a quick look and thought it had already been applied. Yes, please open a PR for it! |
Following issue #83 for episode 5 mplhep
The text was updated successfully, but these errors were encountered: