Skip to content

Conversation

@eyy4
Copy link

@eyy4 eyy4 commented Oct 30, 2019

No description provided.

eyy4 added 2 commits October 30, 2019 17:22
Kept getting permissioning error on rstudio so seeing if this approach will work
Adding community contribution via web
@skyetim
Copy link
Collaborator

skyetim commented Nov 3, 2019

You should create your own branch instead of using the master branch to create the pull request.

@jtr13
Copy link
Owner

jtr13 commented Nov 14, 2019

Can you please respond to the requested changes? This is one of only two remaining pull requests to merge and we'd like to complete the project. If you need help with the instructions please email me.

@skyetim
Copy link
Collaborator

skyetim commented Nov 15, 2019

@vg00164 why do you close this?

@skyetim skyetim reopened this Nov 15, 2019
@VenkataGangireddy
Copy link
Collaborator

VenkataGangireddy commented Nov 15, 2019 via email

@VenkataGangireddy
Copy link
Collaborator

VenkataGangireddy commented Nov 16, 2019 via email

@jtr13
Copy link
Owner

jtr13 commented Nov 20, 2019

What's the status of this? Was it submitted as a separate PR?

@kwj1282
Copy link
Collaborator

kwj1282 commented Nov 20, 2019 via email

@jtr13
Copy link
Owner

jtr13 commented Nov 20, 2019

Thanks @kwj1282. I see now that it wasn't submitted from a non-master branch. @eyy4: Can you meet with me after class tomorrow so I can show you how to fix it and we can wrap this up?

@eyy4
Copy link
Author

eyy4 commented Dec 1, 2019 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants