Update comments

citadel-ai · Feb 19, 2024 · 92139aa · 92139aa
1 parent 6dedc43
commit 92139aa
Showing 1 changed file with 10 additions and 2 deletions.
diff --git a/.github/workflows/pip_install_matrix.yml b/.github/workflows/pip_install_matrix.yml
@@ -27,7 +27,7 @@ jobs:
             os: macos-14
           - python-version: "3.9"
             os: macos-14
-          # TODO: We need to manually install MeCab on Windows to install
+          # TODO: Figure out how to install MeCab on Windows to install
           # LangCheck on Python 3.12 since there are no wheels for `fugashi`
           - python-version: "3.12"
             os: windows-latest
@@ -45,12 +45,16 @@ jobs:
         with:
           python-version: ${{ matrix.python-version }}
 
+      # We need to manually install MeCab on Ubuntu to install LangCheck on
+      # Python 3.12 since there are no wheels for `fugashi`
       - name: Install MeCab on Ubuntu (if necessary)
         if: ${{ matrix.os == 'ubuntu-latest' && matrix.python-version == '3.12' }}
         run: |
           sudo apt-get update
           sudo apt-get install -y mecab libmecab-dev mecab-ipadic-utf8
 
+      # We need to manually install MeCab on macOS to install LangCheck on
+      # Python 3.12 since there are no wheels for `fugashi`
       - name: Install MeCab on macOS (if necessary)
         if: ${{ matrix.os == 'macos-14' && matrix.python-version == '3.12' }}
         run: |
@@ -64,4 +68,8 @@ jobs:
 
       - name: Install Core LangCheck with Optional Dependencies
         run: |
-          pip install .[optional]
+          pip install .[optional]
+
+      - name: Import LangCheck
+        run: |
+          python -c "import langcheck"