search

twjudson / odeproject-public

11 stars 12 forks source link

Typo #1

Closed daverosoff closed 5 years ago

daverosoff commented 5 years ago

In class today we noticed a typo in one of the Sage cells.

twjudson commented 5 years ago

Hi Dave,

Got it! It’s fixed and I pushed it up to the (private) repo. My students are also catching some typos.

Cheers, Tom

On Feb 7, 2019, at 2:10 PM, Dave Rosoff notifications@github.com wrote:

In class today we noticed a typo in one of the Sage cells.

You can view, comment on, or merge this pull request online at:

https://github.com/twjudson/odeproject-public/pull/1

Commit Summary

• Typo File Changes

• M src/firstlook02.xml (2) Patch Links:

• https://github.com/twjudson/odeproject-public/pull/1.patch • https://github.com/twjudson/odeproject-public/pull/1.diff — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or mute the thread.

daverosoff commented 5 years ago

Hi Tom,

Does this mean I should close this PR? I don't think I have access to the private repo. You mentioned adding me to something when we spoke at JMM, but I don't remember seeing any notifications from GitHub. Thanks for the quick response!

Dave

twjudson commented 5 years ago

Yes, close the PR. I will add you to the private repo. What’s your username?

On Feb 7, 2019, at 3:23 PM, Dave Rosoff notifications@github.com wrote:

Hi Tom,

Does this mean I should close this PR? I don't think I have access to the private repo. You mentioned adding me to something when we spoke at JMM, but I don't remember seeing any notifications from GitHub. Thanks for the quick response!

Dave

— You are receiving this because you commented. Reply to this email directly, view it on GitHub, or mute the thread.