Skip to content

Adding community contribution#126

Open
eyy4 wants to merge 3 commits into
jtr13:masterfrom
eyy4:master
Open

Adding community contribution#126
eyy4 wants to merge 3 commits into
jtr13:masterfrom
eyy4:master

Conversation

@eyy4

@eyy4 eyy4 commented Oct 30, 2019

Copy link
Copy Markdown

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

skyetim commented Nov 3, 2019

Copy link
Copy Markdown
Collaborator

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

@jtr13

jtr13 commented Nov 14, 2019

Copy link
Copy Markdown
Owner

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

skyetim commented Nov 15, 2019

Copy link
Copy Markdown
Collaborator

@vg00164 why do you close this?

@skyetim skyetim reopened this Nov 15, 2019
@VenkataGangireddy

VenkataGangireddy commented Nov 15, 2019 via email

Copy link
Copy Markdown
Collaborator

@VenkataGangireddy

VenkataGangireddy commented Nov 16, 2019 via email

Copy link
Copy Markdown
Collaborator

@jtr13

jtr13 commented Nov 20, 2019

Copy link
Copy Markdown
Owner

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

@kwj1282

kwj1282 commented Nov 20, 2019 via email

Copy link
Copy Markdown
Collaborator

@jtr13

jtr13 commented Nov 20, 2019

Copy link
Copy Markdown
Owner

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

eyy4 commented Dec 1, 2019 via email

Copy link
Copy Markdown
Author

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