Skip to content
This repository was archived by the owner on Nov 17, 2023. It is now read-only.

revert DiagrammeR compatibility to 0.9.0#9975

Merged
marcoabreu merged 1 commit intoapache:masterfrom
jeremiedb:graphR2
Mar 5, 2018
Merged

revert DiagrammeR compatibility to 0.9.0#9975
marcoabreu merged 1 commit intoapache:masterfrom
jeremiedb:graphR2

Conversation

@jeremiedb
Copy link
Copy Markdown
Contributor

Follow comments to maintain compatibility with 0.9.0: #9965
attr_theme = NULL is the only material change along the DESCRIPTION requirement. Other changes are just syntax fix.

@jeremiedb jeremiedb requested a review from thirdwing as a code owner March 3, 2018 02:28
@marcoabreu
Copy link
Copy Markdown
Contributor

Is this still compatible with 1.0.0?

@jeremiedb
Copy link
Copy Markdown
Contributor Author

Yes, tested compatibility with DiagrammeR 0.9.0 and 1.0.0.

@marcoabreu marcoabreu merged commit f973fe0 into apache:master Mar 5, 2018
@marcoabreu
Copy link
Copy Markdown
Contributor

@jeremiedb could you please apply these changes to branch 1.0.0 and 1.1.0?

nswamy pushed a commit to nswamy/incubator-mxnet that referenced this pull request Mar 5, 2018
nswamy pushed a commit to nswamy/incubator-mxnet that referenced this pull request Mar 5, 2018
nswamy pushed a commit to nswamy/incubator-mxnet that referenced this pull request Mar 5, 2018
marcoabreu pushed a commit that referenced this pull request Mar 6, 2018
* Fix R build crash in CI (#9965)

* fix viz.graph R

* fix viz.graph R

* revert DiagrammeR compatibility to 0.9.0 (#9975)
@jeremiedb jeremiedb deleted the graphR2 branch March 7, 2018 07:24
@jeremiedb jeremiedb mentioned this pull request Mar 7, 2018
jinhuang415 pushed a commit to jinhuang415/incubator-mxnet that referenced this pull request Mar 30, 2018
rahul003 pushed a commit to rahul003/mxnet that referenced this pull request Jun 4, 2018
zheng-da pushed a commit to zheng-da/incubator-mxnet that referenced this pull request Jun 28, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants