-
Notifications
You must be signed in to change notification settings - Fork 15
ENH: composite widget pipeline #96
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Changes from all commits
b2d9877
fd1b76e
2af82d3
baa28cd
a42684e
2f85811
d6b41c2
65aece1
434d655
2d6b9d8
33fc0c1
cd8eb26
b9b205c
41cd57b
0ca4b74
40a7208
d8cb67c
8ef8224
b0e09d3
b25a942
556b02c
128838e
7fe378f
30d7139
d57d57f
f2bd11c
3b0c16c
cddac2f
96bd725
b487207
1cec097
daf7b01
77ffcc4
90101ac
ef3a6b3
1e6bba1
d765416
fd6e880
208e56d
a08f803
5351089
4573b5d
934f56d
d859a70
cdf7c70
09953d1
ca6a634
6574b48
0b3780d
17b40f1
4d67746
ff6282e
d11f0c8
7a779d7
bec6973
f313a17
58f19ec
650e69b
caffd98
725f47e
5655337
be2989f
431a5da
7fdc872
dd7ecf8
a6a4057
ca3a53f
895aa14
951cacb
dc2a5ce
08e49f3
48d4d57
1dc000c
22acd5f
0914cf5
0cc39d2
03cb5c8
5dc699e
b15f281
6b89e36
0ecb99e
1652da3
656dd78
4364529
6e3d084
5e536b1
b4c13e9
fe1229c
c7f9b99
e2bd95f
c5a4c9d
a3e2af2
d49e7fd
0ec77b3
14176c7
28532fb
d416684
1697982
104f094
dfc2cf4
40025b8
d75dcb5
939a5ff
89ed66c
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
This file was deleted.
This file was deleted.
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. No way uv wants this here instead of in the What a funny little file
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. It's kind of funny, I think the ethos is that this is "main" maintained python version for the repo while the pyproject.toml documents the compatibility range |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1 @@ | ||
| 3.12 |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,11 +1,5 @@ | ||
| include pcdswidgets/_version.py | ||
|
|
||
| include AUTHORS.rst | ||
| include CONTRIBUTING.rst | ||
| include LICENSE.md | ||
| include MANIFEST.in | ||
| include README.md | ||
|
|
||
| include dev-requirements.txt | ||
| include docs-requirements.txt | ||
| include requirements.txt |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,51 @@ | ||
| .PHONY: all build inits venv | ||
|
|
||
| UI_SOURCE := $(wildcard pcdswidgets/ui/*/*/*.ui) | ||
| PY_SOURCE := $(filter-out pcdswidgets/builder/ui/%.py, $(filter-out pcdswidgets/_version.py, $(shell find pcdswidgets -name "*.py"))) | ||
|
|
||
| PY_FORM := $(UI_SOURCE:pcdswidgets/ui/%.ui=pcdswidgets/generated/%_form.py) | ||
| PY_BASE := $(UI_SOURCE:pcdswidgets/ui/%.ui=pcdswidgets/generated/%_base.py) | ||
| PY_MAIN := $(UI_SOURCE:pcdswidgets/ui/%.ui=pcdswidgets/%.py) | ||
|
|
||
| BIN := .venv/bin | ||
| BUILD_CMD := $(BIN)/python -m pcdswidgets.builder.build | ||
| CHECK_FIX := $(BIN)/ruff check --exit-zero --fix --quiet | ||
| FORMAT := $(BIN)/ruff format --quiet | ||
|
|
||
| # We need to update the venv before doing any step and after doing all of them | ||
| # The order matters here, except the py files in the build target could be done in any order | ||
| all: | ||
| $(MAKE) venv | ||
| $(MAKE) build | ||
| $(MAKE) inits | ||
| $(MAKE) pyproject.toml | ||
| $(MAKE) venv | ||
|
|
||
| build: $(PY_FORM) $(PY_BASE) $(PY_MAIN) inits | ||
|
|
||
| # Need to re-run form and base if the ui file is updated | ||
| $(PY_FORM): pcdswidgets/generated/%_form.py: pcdswidgets/ui/%.ui | ||
| $(BUILD_CMD) uic $^ | ||
| $(CHECK_FIX) $@ | ||
| $(FORMAT) $@ | ||
|
|
||
| $(PY_BASE): pcdswidgets/generated/%_base.py: pcdswidgets/ui/%.ui | ||
| $(BUILD_CMD) base $^ | ||
| $(CHECK_FIX) $@ | ||
| $(FORMAT) $@ | ||
|
|
||
| # Only run if the target is missing: user can edit these | ||
| $(PY_MAIN): | ||
| $(BUILD_CMD) main $(@:pcdswidgets/%.py=pcdswidgets/ui/%.ui) | ||
| $(CHECK_FIX) $@ | ||
| $(FORMAT) $@ | ||
|
|
||
| inits: | ||
| $(BIN)/python -m pcdswidgets.builder.inits | ||
|
|
||
| # Rerun if any python file is updated | ||
| pyproject.toml: $(PY_SOURCE) | ||
| $(BIN)/python -m pcdswidgets.builder.entrypoint_finder | ||
|
|
||
| venv: | ||
| ./build_local_venv.sh |
Uh oh!
There was an error while loading. Please reload this page.