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

replace d by plotattributes inside recipes #1116

Merged
merged 2 commits into from
Oct 2, 2017

Conversation

jw3126
Copy link
Contributor

@jw3126 jw3126 commented Sep 29, 2017

See #1107. Please tag a new release after merging.

@jw3126 jw3126 closed this Sep 30, 2017
@jw3126 jw3126 reopened this Sep 30, 2017
@jw3126
Copy link
Contributor Author

jw3126 commented Sep 30, 2017

There is some PyPlot related CI failure. Is this related to my changes?

@mkborregaard
Copy link
Member

No, that happened before. We'll inspect manually to see whether the gr tests pass.

@jw3126
Copy link
Contributor Author

jw3126 commented Oct 2, 2017

So all gr tests pass on Travis. On some AppVeyor setups all tests fail. My impression is that the failures are unrelated to this PR. Do I see this correctly? Should I change something or is this ready for merge?

@mkborregaard
Copy link
Member

Ready :-)

@mkborregaard mkborregaard merged commit a80c7ad into JuliaPlots:master Oct 2, 2017
@jw3126
Copy link
Contributor Author

jw3126 commented Oct 2, 2017

Is it possible to tag a new release of Plots in the next days? So I can PR downstream packages?

@mkborregaard
Copy link
Member

Yes, I'm doing it

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.

2 participants