/matplotlib

Remove shebang lines from all examples. [MEP12] #6342

Merged
merged 1 commit into from Apr 29, 2016

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
MEP12
Projects
None yet
Milestone
  v2.0.0
5 participants
@TrishGillett @tacaswell @dopplershift @mdboom @QuLogic
@TrishGillett
Contributor

TrishGillett commented Apr 28, 2016

I'm planning a hacknight soon with my local PyLadies group to contribute some work on example clean up, but deleting shebang lines seemed like one aspect of MEP12 that could be easily done in bulk ahead of time. I believe this gets rid of every single shebang in the examples folder. You might even say it takes care of... the whole shebang. 😎

This commit also includes a handful of other tweaks to file headers. Nothing exhaustive, just things I noticed while the files were open.
@TrishGillett TrishGillett Remove shebang lines from all examples. [MEP12]

Some checks were not successful

1 failing and 2 successful checks
continuous-integration/appveyor/pr — AppVeyor build failed
Details
continuous-integration/travis-ci/pr — The Travis CI build passed
Details
coverage/coveralls — Coverage remained the same at 69.658%
Details
13c3191

mdboom added the needs_review label Apr 28, 2016

@tacaswell
Owner

tacaswell commented Apr 29, 2016

👍 Appveyor looks like downloading miniconda is flaky.

@tacaswell tacaswell merged commit 9533cc6 into matplotlib:master Apr 29, 2016
2 of 3 checks passed

2 of 3 checks passed

continuous-integration/appveyor/pr AppVeyor build failed
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage remained the same at 69.658%
Details

tacaswell removed the needs_review label Apr 29, 2016

QuLogic added this to the 2.1 (next point release) milestone Apr 29, 2016

QuLogic added the MEP12 label Apr 29, 2016

@dopplershift
Contributor

dopplershift commented Apr 29, 2016

FYI, AppVeyor has a miniconda install already on the machines now.

@tacaswell tacaswell added a commit that referenced this pull request Nov 11, 2016

@tacaswell tacaswell Merge pull request #6342 from discardthree/mep12-hacknight-prep 
DOC: Remove shebang lines from all examples. [MEP12]

Conflicts:
	examples/pylab_examples/date_demo1.py
	  - removed since the source PR
	examples/pylab_examples/date_demo2.py
	  - removed since the source PR
	examples/pylab_examples/finance_demo.py
	  - removed since the source PR
	examples/pylab_examples/mri_with_eeg.py
	  - resolved in favor of current text
 c922fcc
@tacaswell
Owner

tacaswell commented Nov 11, 2016

backporting this to v2.x to make backporting other docs changes easier.

backported to v2.x as c922fcc

@tacaswell tacaswell modified the milestone: 2.0.1 (next bug fix release), 2.1 (next point release) Nov 11, 2016

@QuLogic QuLogic modified the milestone: 2.0.1 (next bug fix release), 2.0 (style change major release) Dec 7, 2016

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