Merge pull request #289 from gaphor/remove-ci-workaround-for-console

Remove CI workaround for console plugin
gaphor · Apr 10, 2020 · 31f550b · 31f550b
2 parents 4eaf0c8 + 8cefcb9
commit 31f550b
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 15 deletions.
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -38,15 +38,12 @@ jobs:
     needs: lint
     runs-on: ubuntu-latest
     if: "!contains(github.event.head_commit.message, 'skip ci')"
-    strategy:
-      matrix:
-        py_ver: [3.7, 3.8]
     steps:
       - uses: actions/checkout@v1
       - name: Set up Python
         uses: actions/setup-python@v1
         with:
-          python-version: ${{ matrix.py_ver }}
+          python-version: 3.8
       - name: Use Python Dependency Cache
         uses: actions/cache@v1.0.3
         with:

diff --git a/gaphor/plugins/console/console.py b/gaphor/plugins/console/console.py
@@ -10,21 +10,11 @@
 import os
 import pydoc
 import sys
+from rlcompleter import Completer
 from typing import Dict, List
 
 from gi.repository import Gdk, GLib, Gtk, Pango
 
-if os.getenv("GITHUB_REF"):
-    # Work around issues in CI with loading rlcompleter
-    # https://github.com/gaphor/gaphor/issues/245
-    class Completer:
-        pass
-
-
-else:
-    from rlcompleter import Completer
-
-
 if __name__ == "__main__":
     import gi