Skip to content

Commit

Permalink
Merge pull request #2843 from tomjakubowski/dmv
Browse files Browse the repository at this point in the history
sdist licensing follow ups
texodus authored Nov 13, 2024

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
2 parents 452bfb0 + 053abcd commit 632c051
Showing 9 changed files with 362 additions and 21 deletions.
13 changes: 13 additions & 0 deletions .github/workflows/build.yaml
Original file line number Diff line number Diff line change
@@ -110,6 +110,7 @@ jobs:
rust/perspective-client/docs/
rust/perspective-js/src/ts/
rust/perspective-viewer/src/ts/
rust/perspective-python/LICENSE_THIRDPARTY_cargo.yml
# ,-,---. . . ,-_/ .---. .
# '|___/ . . . | ,-| ' | ,-. . , ,-. \___ ,-. ,-. . ,-. |-
@@ -485,6 +486,11 @@ jobs:
name: perspective-js-dist
path: .

- uses: actions/download-artifact@v4
with:
name: perspective-metadata
path: rust/

- uses: actions/download-artifact@v4
with:
name: perspective-python-dist-${{ matrix.arch }}-${{ matrix.os }}-${{ matrix.python-version }}
@@ -707,6 +713,13 @@ jobs:
shell: bash
run: python -m pip install -vv ./perspective*.tar.gz

- name: Verify licenses are installed
shell: bash
run: |
pip show -f perspective-python > wheel_installed_files.txt
grep licenses/LICENSE.md wheel_installed_files.txt
grep licenses/LICENSE_THIRDPARTY_cargo.yml wheel_installed_files.txt
- name: Verify labextension
shell: bash
run: |
3 changes: 2 additions & 1 deletion .gitignore
Original file line number Diff line number Diff line change
@@ -247,7 +247,8 @@ rust/perspective-server/cmake
.pyodide-*/
rust/perspective-python/*.data
rust/perspective-python/PKG-INFO
# rust/perspective-python/perspective.data/data/share/jupyter/labextensions/@finos/perspective-jupyterlab/package.json
rust/perspective-python/LICENSE.md
rust/perspective-python/LICENSE_*
rust/perspective-viewer/docs/exprtk.md
rust/perspective-server/docs/lib_gen.md
rust/perspective-client/docs/expression_gen.md
Loading

0 comments on commit 632c051

Please sign in to comment.