Skip to content
This repository has been archived by the owner on Jan 24, 2025. It is now read-only.

Pull request tutorial submission #13

Open
tyjh2015 opened this issue Dec 10, 2014 · 9 comments
Open

Pull request tutorial submission #13

tyjh2015 opened this issue Dec 10, 2014 · 9 comments

Comments

@tyjh2015
Copy link
Contributor

This is my submission

@denten
Copy link
Member

denten commented Dec 10, 2014

Heya, this is just an issue. Where's the pull request?

@tyjh2015
Copy link
Contributor Author

I have an open pull request from the same repo (last week's about pizza.)
Can you accept that one and get this week's tutorial? Thanks

On Tuesday, December 9, 2014, Dennis Tenen [email protected] wrote:

Heya, this is just an issue. Where's the pull request?


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

@denten
Copy link
Member

denten commented Dec 13, 2014

You should be all set. Check the main repo to see that the changes took.

On Tue, Dec 9, 2014 at 10:09 PM, tyjh2015 [email protected] wrote:

This is my submission


Reply to this email directly or view it on GitHub
#13.

@tyjh2015
Copy link
Contributor Author

Hi Dennis,
I don't see my submission in the main repo.

Best,
Yee

On Fri, Dec 12, 2014 at 7:26 PM, Dennis Tenen [email protected]
wrote:

You should be all set. Check the main repo to see that the changes took.

On Tue, Dec 9, 2014 at 10:09 PM, tyjh2015 [email protected]
wrote:

This is my submission


Reply to this email directly or view it on GitHub
#13.


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

@denten
Copy link
Member

denten commented Dec 14, 2014

Yee, I just merged the next batch of new pull requests. Please check again.

@tyjh2015
Copy link
Contributor Author

Hi Dennis,

My user name is tyjh, easily confused with tjh2145, which is what i see was
just merged.
here's the original:

#6 (comment)

On Sun, Dec 14, 2014 at 5:29 PM, Dennis Tenen [email protected]
wrote:

Yee, I just merged the next batch of new pull requests. Please check again.


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

@denten
Copy link
Member

denten commented Dec 14, 2014

I merged them all! The pizza one I just closed without merging. Reopened
and merged now.

On Sun, Dec 14, 2014 at 6:08 PM, tyjh2015 [email protected] wrote:

Hi Dennis,

My user name is tyjh, easily confused with tjh2145, which is what i see
was
just merged.
here's the original:

#6 (comment)

On Sun, Dec 14, 2014 at 5:29 PM, Dennis Tenen [email protected]
wrote:

Yee, I just merged the next batch of new pull requests. Please check
again.


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


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

@tyjh2015
Copy link
Contributor Author

Cool, thanks!
Do you have time tomorrow afternoon to meet briefly?

Best,
Yee

On Sun, Dec 14, 2014 at 6:16 PM, Dennis Tenen [email protected]
wrote:

I merged them all! The pizza one I just closed without merging. Reopened
and merged now.

On Sun, Dec 14, 2014 at 6:08 PM, tyjh2015 [email protected]
wrote:

Hi Dennis,

My user name is tyjh, easily confused with tjh2145, which is what i see
was
just merged.
here's the original:

#6 (comment)

On Sun, Dec 14, 2014 at 5:29 PM, Dennis Tenen [email protected]

wrote:

Yee, I just merged the next batch of new pull requests. Please check
again.


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


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


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

@denten
Copy link
Member

denten commented Dec 15, 2014

I am all booked up. Give me a call (415-215-3315) before six or catch me on
gMail chat!

On Sun, Dec 14, 2014 at 6:32 PM, tyjh2015 [email protected] wrote:

Cool, thanks!
Do you have time tomorrow afternoon to meet briefly?

Best,
Yee

On Sun, Dec 14, 2014 at 6:16 PM, Dennis Tenen [email protected]
wrote:

I merged them all! The pizza one I just closed without merging. Reopened
and merged now.

On Sun, Dec 14, 2014 at 6:08 PM, tyjh2015 [email protected]
wrote:

Hi Dennis,

My user name is tyjh, easily confused with tjh2145, which is what i
see
was
just merged.
here's the original:

#6 (comment)

On Sun, Dec 14, 2014 at 5:29 PM, Dennis Tenen <
[email protected]>

wrote:

Yee, I just merged the next batch of new pull requests. Please check
again.


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


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


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


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

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants