swcarpentry / DEPRECATED-bc

DEPRECATED: This repository is now frozen - please see individual lesson repositories.
Other
299 stars 383 forks source link

Some minor edits to the loops lesson #896

Closed chinchliff closed 7 years ago

chinchliff commented 9 years ago

I added a challenge problem that introduces while loops--these don't seem to be critical to the lesson material but they are an important element of programming and it seems useful to at least mention them here. I also reworded some of the explanation/discussion of mutable vs. immutable variables, in an effort to improve clarity. Finally, I made a couple of other (very minor) changes to some examples.

rgaiacs commented 9 years ago

@chinchliff Thanks for this. The file novice/python/03-loop.md is generated using

$ make ipynb

Do you mind

$ git remote add upstream git@github.com:swcarpentry/bc.git
$ git fetch upstream
$ git checkout upstream/gh-pages
$ git checkout -b improve-python-loop
$ make ipynb
$ git commit -a
$ git push origin improve-python-loop

Why am I requesting this? (1) To you learn a little about branches, (2) to avoid the commit where you edit novice/python/03-loop.md by hand and (2) to avoid the various "minor formatting edit" commits. If you need any help just ask.

gvwilson commented 9 years ago

Hi @chinchliff,

We can make this even easier - if you're willing to wait a couple of days, you'll be able to make your change in Markdown only (no IPython Notebook) in the new Python lesson repo - we're that close to merging it. Please stay tuned... Or if you want to go ahead and merge here, please do - we're happy to get content either way.

cheers, Greg

chinchliff commented 9 years ago

Hi Greg, Raniere,

I'm happy to wait a couple of days. If you could give me a heads up when the new repo is ready I'll make a new pull request there with the changes.

On Monday, November 24, 2014, Greg Wilson notifications@github.com wrote:

Hi @chinchliff,

We can make this even easier - if you're willing to wait a couple of days, you'll be able to make your change in Markdown only (no IPython Notebook) in the new Python lesson repo - we're that close to merging it. Please stay tuned... Or if you want to go ahead and merge here, please do - we're happy to get content either way.

cheers, Greg

— Reply to this email directly or view it on GitHub https://github.com/swcarpentry/bc/pull/896#issuecomment-64300988.