-
Notifications
You must be signed in to change notification settings - Fork 62
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
Update plot.jl to account for line style #131
Conversation
Codecov Report
@@ Coverage Diff @@
## master #131 +/- ##
==========================================
+ Coverage 33.26% 33.39% +0.12%
==========================================
Files 9 9
Lines 526 527 +1
==========================================
+ Hits 175 176 +1
Misses 351 351
Continue to review full report at Codecov.
|
It seems like there are some pull requests pending; do you see that they will be investigated and completed in the foreseable future? |
@simonschoelly, did this PR fall through the cracks? |
oops, I messed something up... |
I tried to rebase the PR and add a commit withsome documentation about it, but that didn't worked as expected. Maybe I should try reopening a PR. |
Ok, I think I understand what happened: the PR was on master, and I already add a master branch created. so when I fetched it, it renamed the branch, and when I pushed, it pushed my master instead of the modifications. Sorry about that, I will make a new PR. |
new PR: #176 |
|
I think this is what I did, but I pushed the master instead of the PR branch |
Hi,
This commit resolves #127
Regards,
Davide