Skip to content

Commit

Permalink
V0.1.x (#649)
Browse files Browse the repository at this point in the history
* fix for LIN-340. adding a publish workflow. it should build and push to test pypi for now. It only gets triggered after successful python workflow run so that only after tests pass will it trigger.

* enable python workflow on release branch

* whitespace commit

* do not check out recursively. not needed for builds i think
  • Loading branch information
lionsardesai committed May 18, 2022
1 parent f04aaa1 commit 604c1db
Show file tree
Hide file tree
Showing 6 changed files with 47 additions and 5 deletions.
2 changes: 2 additions & 0 deletions .cspell/custom-dictionary-workspace.txt
Original file line number Diff line number Diff line change
Expand Up @@ -15,13 +15,15 @@ kwargs
linea
lineabuiltins
lineapy
listify
maxdepth
nbconvert
nbformat
nbsphinx
nbval
NBVAL_IGNORE_OUTPUT
networkx
orms
psycopg2
Pydantic
scipy
Expand Down
36 changes: 36 additions & 0 deletions .github/workflows/publish.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,36 @@
name: Publish

on:
workflow_run:
workflows: [Python]
types: [completed]
branches:
- "v[0-9]+.[0-9]+.x"

jobs:
build-n-publish:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v2
with:
lfs: true
- name: Set up Python 3.9
uses: actions/setup-python@v2
with:
python-version: 3.9
- name: Install dependencies
run: |
python setup.py install && rm -rf build dist
- name: Build Wheels
run: |
python setup.py sdist bdist_wheel
- name: Check build
run: |
twine check dist/*
- name: Publish package
if: github.event_name == 'push' && startsWith(github.ref, 'refs/tags')
uses: pypa/gh-action-pypi-publish@release/v1
with:
user: __token__
password: ${{ secrets.TEST_PYPI_API_TOKEN }}
repository_url: https://test.pypi.org/legacy/
4 changes: 3 additions & 1 deletion .github/workflows/python-app.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,9 @@ env:

on:
push:
branches: [main]
branches:
- main
- "v[0-9]+.[0-9]+.x"
paths:
- "**.py"
- "**.ipynb"
Expand Down
6 changes: 4 additions & 2 deletions lineapy/graph_reader/apis.py
Original file line number Diff line number Diff line change
Expand Up @@ -192,8 +192,10 @@ class LineaCatalog:
"""

"""
DEV NOTE:
- The export is pretty limited right now and we should expand later.
.. note::
- The export is pretty limited right now and we should expand later.
"""

def __init__(self, db):
Expand Down
1 change: 1 addition & 0 deletions lineapy/plugins/task.py
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,7 @@ class TaskGraph(object):
.. note::
- If we only support Python 3.9+, we prefer to use graphlib in standard
library instead of networkx for graph operation.
Expand Down
3 changes: 1 addition & 2 deletions lineapy/transformer/node_transformer.py
Original file line number Diff line number Diff line change
Expand Up @@ -105,8 +105,7 @@ def transform(

class NodeTransformer(ast.NodeTransformer):
"""
NOTE
----
.. note::
- Need to be careful about the order by which these calls are invoked
so that the transformation do not get called more than once.
Expand Down

0 comments on commit 604c1db

Please sign in to comment.