Skip to content

Conversation

pravsingh
Copy link

Ref: #32

Please review. I'll incorporate your feedbacks in my next commit.

FYI @bdonlan

@bdonlan
Copy link
Contributor

bdonlan commented Nov 8, 2017

Unfortunately it looks like this branch currently conflicts and can't be merged. There are also some other changes in e.g. PR #40 that will touch those same files soon, so it's best not to rebase just yet.

That being said, I'm thinking the better approach here would be to move them off to a separate repo - I don't want to have the version number in multiple places and add more levels of directories to the source tree if we can avoid it. I'll look into getting a new repo set up on our side.

@bdonlan
Copy link
Contributor

bdonlan commented Dec 7, 2017

Closing this PR for now, we'll track the progress in #32 until we can finish setting up the new examples repo.

@bdonlan bdonlan closed this Dec 7, 2017
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