Skip to content
This repository was archived by the owner on Sep 4, 2018. It is now read-only.

Add Codefellows phone number#1

Closed
charlessipe wants to merge 2 commits intoOperationCode:masterfrom
charlessipe:master
Closed

Add Codefellows phone number#1
charlessipe wants to merge 2 commits intoOperationCode:masterfrom
charlessipe:master

Conversation

@charlessipe
Copy link
Copy Markdown
Contributor

Trying a test of a pull request

@charlessipe
Copy link
Copy Markdown
Contributor Author

I added some online code schools to the code schools page (Bloc, The Firehose Project, Tealeaf Academy).

@davidmolina
Copy link
Copy Markdown
Member

@charlessipe - Thanks so much for your contribution! Can you please take out those .DS_store files, move the others to "Learning Resources" as they are online resources only. Also, remove number. We'd have to add numbers to all the services. Those might change more periodically than the URL. Thanks!

@davidmolina
Copy link
Copy Markdown
Member

@nanoxd

@davidmolina
Copy link
Copy Markdown
Member

Hmm. Why can't I merge this request?

@emckenna
Copy link
Copy Markdown
Contributor

says merge conflict? there are some DS_STORE files in the pr as well.

@emckenna
Copy link
Copy Markdown
Contributor

I think @charlessipe needs to make a branch from his master?, then submit that pull request to merge to upstream master. Not sure if that is required or not.

@davidmolina
Copy link
Copy Markdown
Member

Think that's right. Possible @charlessipe https://github.com/charlessipe?

On Tue, Jul 21, 2015 at 6:57 AM, Eric Mckenna [email protected]
wrote:

I think @charlessipe https://github.com/charlessipe needs to make a
branch from his master?, then submit that pull request to merge to upstream
master. Not sure if that is required or not.


Reply to this email directly or view it on GitHub
#1 (comment)
.

David
t. @davidcmolina https://twitter.com/davidcmolina
http://davidmolina.github.io/

@rickr
Copy link
Copy Markdown
Contributor

rickr commented Mar 21, 2016

Hi @charlessipe,

Looks like this PR is over a year old. Is the info still relevant? If it is can you rebase onto master and resubmit?

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants