Skip to content

Conversation

makai410
Copy link
Contributor

Blocked on #104

Copy link
Contributor

@celinval celinval left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we merge this PR now if we comment out the rustc_public dependency from demo/Cargo.toml and add an extern crate rustc_public for now?

@makai410
Copy link
Contributor Author

It works, but I think #104 should be good to merge as I've rebased it and things are in the rustc_public folder now.

Btw my commits are ready for review, so after that PR is merged I'll open a series of prs. maybe we could wait for it then?

@makai410
Copy link
Contributor Author

or is there any blocker on #104? If so, it makes sense to do that for now

@celinval
Copy link
Contributor

I do think there might be a blocker. Let me post it there

@makai410
Copy link
Contributor Author

okay done

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