Merge ~racb/git-ubuntu:unpin-pylint into git-ubuntu:master

Proposed by Robie Basak
Status: Merged
Merged at revision: 85bef3c41e85d02501ec3028e4bedb882526231b
Proposed branch: ~racb/git-ubuntu:unpin-pylint
Merge into: git-ubuntu:master
Diff against target: 78 lines (+9/-18)
4 files modified
gitubuntu/source-package-whitelist.txt (+4/-2)
setup.py (+1/-0)
snap-wrappers/wrappers/git-ubuntu-self-test (+4/-2)
snap/snapcraft.yaml (+0/-14)
Reviewer Review Type Date Requested Status
Andreas Hasenack Approve
Server Team CI bot continuous-integration Approve
Review via email: mp+359847@code.launchpad.net

Commit message

Make Jenkins happy

To post a comment you must log in.
Revision history for this message
Robie Basak (racb) wrote :

This MP is really only for three commits. The remainder are a Launchpad bug.

Revision history for this message
Server Team CI bot (server-team-bot) wrote :

PASSED: Continuous integration, rev:85bef3c41e85d02501ec3028e4bedb882526231b
https://jenkins.ubuntu.com/server/job/git-ubuntu-ci/81/
Executed test runs:
    SUCCESS: VM Setup
    SUCCESS: Build
    SUCCESS: Unit Tests
    IN_PROGRESS: Declarative: Post Actions

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/server/job/git-ubuntu-ci/81/rebuild

review: Approve (continuous-integration)
Revision history for this message
Andreas Hasenack (ahasenack) wrote :

Looks good, +1

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/gitubuntu/source-package-whitelist.txt b/gitubuntu/source-package-whitelist.txt
2index 5e20d9e..f966474 100644
3--- a/gitubuntu/source-package-whitelist.txt
4+++ b/gitubuntu/source-package-whitelist.txt
5@@ -891,9 +891,11 @@ zerofree
6 zlib
7 zope.interface
8 # Requested by ahasenack
9-skytools3
10-pmdk
11+lmdb
12 ndctl
13+pgaudit
14+pmdk
15+skytools3
16 # Requested by mwhudson
17 cargo
18 rustc
19diff --git a/setup.py b/setup.py
20index 9aa66e5..f97740c 100644
21--- a/setup.py
22+++ b/setup.py
23@@ -16,6 +16,7 @@ setup(name='gitubuntu',
24 'pygit2==0.24.2',
25 'launchpadlib==1.10.6',
26 'petname',
27+ 'pylint',
28 'setuptools',
29 'keyring<16', # https://github.com/jaraco/keyrings.alt/issues/28
30 'keyrings.alt',
31diff --git a/snap-wrappers/wrappers/git-ubuntu-self-test b/snap-wrappers/wrappers/git-ubuntu-self-test
32index f57cde7..76b6074 100755
33--- a/snap-wrappers/wrappers/git-ubuntu-self-test
34+++ b/snap-wrappers/wrappers/git-ubuntu-self-test
35@@ -1,6 +1,8 @@
36 #!/snap/core/current/bin/bash
37
38-python3 -m pylint --ignored-modules=pygit2 -E $SNAP/lib/python3.6/site-packages/gitubuntu/
39+cd $SNAP/lib/python3.6/site-packages
40+
41+python3 -m pylint --ignored-modules=pygit2,py.path -E gitubuntu/
42 PYLINT_RC=$?
43
44 if [ $PYLINT_RC == 0 ]; then
45@@ -8,7 +10,7 @@ if [ $PYLINT_RC == 0 ]; then
46 fi
47
48 TMPFILE=$(mktemp)
49-COVERAGE_FILE=$TMPFILE python3 -m pytest -p no:cacheprovider --cov=$SNAP/lib/python3.6/site-packages/gitubuntu $SNAP/lib/python3.6/site-packages/gitubuntu/*
50+COVERAGE_FILE=$TMPFILE python3 -m pytest -p no:cacheprovider --cov=gitubuntu gitubuntu/*
51 PYTEST_RC=$?
52 unlink $TMPFILE
53
54diff --git a/snap/snapcraft.yaml b/snap/snapcraft.yaml
55index e9a74d6..eacf05b 100644
56--- a/snap/snapcraft.yaml
57+++ b/snap/snapcraft.yaml
58@@ -353,20 +353,6 @@ parts:
59 - -usr/bin/xzless
60 - -usr/bin/xzmore
61 after: [gnupg2]
62- pylint:
63- plugin: python
64- python-version: python3
65- # from https://github.com/PyCQA/pylint/archive/pylint-1.7.2.tar.gz
66- source: https://people.canonical.com/~rbasak/git-ubuntu/pylint-1.7.2.tar.gz
67- source-type: tar
68- source-checksum: sha384/64b4f1fb40d80eb8cfbaa7aa9bf603e8194a882c2689b9a56157216244d834b0c9998467aeb093c8a3b478a10bfe3be6
69- stage:
70- - -usr/bin/py*
71- - -usr/bin/2to3*
72- - -usr/share/*python*
73- - -usr/lib/python3.5
74- - -usr/lib/python3.6
75- after: [python3]
76 git-ubuntu:
77 plugin: python
78 python-version: python3

Subscribers

People subscribed via source and target branches