Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add articles to R Programming page #130

Merged
merged 7 commits into from
May 27, 2017

Conversation

lgreski
Copy link
Contributor

@lgreski lgreski commented May 20, 2017

No description provided.

@seankross
Copy link
Member

@lgreski I don't believe there is a capstone page.

@lgreski
Copy link
Contributor Author

lgreski commented May 23, 2017 via email

@seankross
Copy link
Member

Would you mind taking the link out until the link is live? (Or I can just wait until you add the commit.)

@lgreski
Copy link
Contributor Author

lgreski commented May 23, 2017 via email

@lgreski
Copy link
Contributor Author

lgreski commented May 23, 2017

Added the capstone.md page.

@seankross
Copy link
Member

@lgreski Is this ready to merge?

@lgreski
Copy link
Contributor Author

lgreski commented May 27, 2017 via email

@seankross
Copy link
Member

Awesome, thanks Len!

@seankross seankross merged commit 1e2c829 into DataScienceSpecialization:master May 27, 2017
@lgreski
Copy link
Contributor Author

lgreski commented May 27, 2017

You're welcome. I apologize about the initial issue. I didn't realize that once I'd submitted a pull request, that subsequent commits to my version of the repository are automatically included in the pull. I thought I had submitted a complete package and was waiting for it to be completed before submitting the updated index and capstone pages. I"m usually pretty conscientious about making my pull requests complete units of work.

@lgreski
Copy link
Contributor Author

lgreski commented May 27, 2017

@seankross looks good, Capstone page is available from the home page, correctly brings up capstone.md, and all links on capstone.md bring up the correct pages.

@seankross
Copy link
Member

Don't worry about it! One suggestion: you could create a separate branch for each "unit of work" and then initiate the PR from that branch. That way your commits to other branches wouldn't show up.

@lgreski
Copy link
Contributor Author

lgreski commented May 27, 2017

That's a good point. Usually when I contribute content to the DSS repository the units of work are smaller, so no need for a separate branch.

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