[MRG] Deprecate montage param from all EEG raw readers

[agramfort]

alternative to #6526

[agramfort]

crap my master was behind :( :(

@massich I tried to the job with what i consider a dumb approach....

[codecov]

Codecov Report

Merging #6534 into master will decrease coverage by <.01%.
The diff coverage is 95.23%.

@@            Coverage Diff             @@
##           master    #6534      +/-   ##
==========================================
- Coverage   89.34%   89.33%   -0.01%     
==========================================
  Files         416      416              
  Lines       74865    74889      +24     
  Branches    12341    12344       +3     
==========================================
+ Hits        66885    66905      +20     
- Misses       5137     5138       +1     
- Partials     2843     2846       +3

[lgtm-com]

This pull request introduces 1 alert when merging 05217a8 into 201ce50 - view on LGTM.com

new alerts:

• 1 for Mismatch between signature and use of an overridden method

[agramfort]

we have the same error on master now https://travis-ci.org/mne-tools/mne-python/jobs/555779112

not sure about the circle failure though

[larsoner]

/home/circleci/project/doc/whats_new.rst:75: WARNING: py:func reference target not found: mne.io.read_epochs_eeglab

[lgtm-com]

This pull request introduces 1 alert when merging 53bf894 into 7d64af3 - view on LGTM.com

new alerts:

• 1 for Mismatch between signature and use of an overridden method

[massich]

This PR only deprecates that montage has to be None. It does not deprecate the montage parameter itself.

[lgtm-com]

This pull request introduces 1 alert when merging 92c1061 into 964ea6c - view on LGTM.com

new alerts:

• 1 for Mismatch between signature and use of an overridden method

[larsoner]

Usually in the messages we say explicitly when it will be removed, in this case 0.20. otherwise LGTM +1 for merge

[agramfort]

done

…

[lgtm-com]

This pull request introduces 1 alert when merging 5ffe301 into 26231ff - view on LGTM.com

new alerts:

• 1 for Mismatch between signature and use of an overridden method

[massich]

+1, once travis is green I'll fix the whatsnew and merge it.

[lgtm-com]

This pull request introduces 1 alert when merging 56986b7 into 2b11463 - view on LGTM.com

new alerts:

• 1 for Mismatch between signature and use of an overridden method

[massich]

we should not be able to create info with montage shuold we?
https://github.com/mne-tools/mne-python/blob/master/mne/io/meas_info.py#L1586

[agramfort]

it should be possible but accepting a Digitization instance should be aimed for.