Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Python minimum version 3.4.4 #2612

Merged
merged 4 commits into from
Aug 16, 2018
Merged

Python minimum version 3.4.4 #2612

merged 4 commits into from
Aug 16, 2018

Conversation

jayvdb
Copy link
Member

@jayvdb jayvdb commented Jul 17, 2018

Closes #2611
Closes #2488

@gitmate-bot
Copy link
Collaborator

Comment on 44012a9.

Shortlog of the HEAD commit contains 52 character(s). This is 2 character(s) longer than the limit (52 > 50).

Origin: GitCommitBear, Section: all.commit.

@jayvdb
Copy link
Member Author

jayvdb commented Jul 17, 2018

Demo of the py27 and py33 jobs: https://travis-ci.org/jayvdb/coala-bears/builds/404723796
That is commit jayvdb@8229d58 which has parent 807b537, same commit as this PR.

@jayvdb
Copy link
Member Author

jayvdb commented Jul 17, 2018

The GitMate PR error is because of the revert commit, 44012a9 , which goes over the length limit because #2241 hasnt been finished.

@gitmate-bot
Copy link
Collaborator

Comment on 44012a9.

Shortlog of the HEAD commit contains 52 character(s). This is 2 character(s) longer than the limit (52 > 50).

Origin: GitCommitBear, Section: all.commit.

@jayvdb
Copy link
Member Author

jayvdb commented Jul 17, 2018

Demo https://travis-ci.org/jayvdb/coala-bears/builds/404768441
is commit jayvdb@3728c6d with parent jayvdb@ace719b which is the same as this PR

Copy link
Member

@sangamcse sangamcse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@gitmate-bot
Copy link
Collaborator

Comment on cc98e23.

Shortlog of the HEAD commit contains 52 character(s). This is 2 character(s) longer than the limit (52 > 50).

Origin: GitCommitBear, Section: all.commit.

jayvdb added 2 commits August 15, 2018 07:15
Many of the bear dependencies are unable to be installed
on older unsupported versions of Python. The unsupported
tests objective is to verify that irrespective of any
other failure to install, coala-bears has its own check
to prevent running on older versions. Hence skipping the
installation of bear dependencies ensures the check
still works even if other packages dont break first.

Closes coala#2488
Related to coala#2611
coala updated its minimum version to 3.4.4 a
while ago and bears was not also updated.

Adds tests to cover assert_supported_version,
so that the pragma no cover can be removed
and setup.py does not need to be run by coverage.

Related to coala/coala#4350
@jayvdb
Copy link
Member Author

jayvdb commented Aug 15, 2018

Instead of setting coverage to run on all files
and omitting a list of files, replace

    source = .

with

    source = bears

This brings bears into line with setup.cfg.jj2

Related to https://gitlab.com/coala/mobans/issues/24
@jayvdb jayvdb force-pushed the min-py344 branch 2 times, most recently from 39273c1 to d07dde4 Compare August 15, 2018 12:50
This brings the bear script into line with the
moban template.

Also connect check_moban.sh with the moban asset.

Related to https://gitlab.com/coala/mobans/issues/65
@jayvdb
Copy link
Member Author

jayvdb commented Aug 15, 2018

Fresh verification build https://travis-ci.org/jayvdb/coala-bears/builds/416374555

To verify yourself, cherry-pick the following and run on a fork

diff --git a/.travis.yml b/.travis.yml
index c3e4fb4..80b0b6c 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -24,7 +24,7 @@ stages:
   - test
   - moban
   - name: unsupported
-    if: branch = master AND type = push
+    if: type != pull_request
 
 .check_moban: &check_moban
   <<: *disable_global

@ksdme
Copy link
Member

ksdme commented Aug 16, 2018

LGTM.

@jayvdb
Copy link
Member Author

jayvdb commented Aug 16, 2018

ack 5ed8b8c 5379f4e aa60885 d07dde4

@jayvdb
Copy link
Member Author

jayvdb commented Aug 16, 2018

@gitmate-bot ff

@gitmate-bot
Copy link
Collaborator

Hey! I'm GitMate.io! This pull request is being fastforwarded automatically. Please DO NOT push while fastforward is in progress or your changes would be lost permanently ⚠️

@gitmate-bot
Copy link
Collaborator

Automated fastforward with GitMate.io was successful! 🎉

@gitmate-bot gitmate-bot merged commit d07dde4 into coala:master Aug 16, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

4 participants