Skip to content
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.

Sign up for GitHub

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

Jump to bottom

refactor: accommodate recent pymake changes #26

Merged
merged 7 commits into from
Feb 18, 2024
Merged

refactor: accommodate recent pymake changes #26

Changes from 1 commit
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
59bd345
refactor: accommodate recent pymake changes
wpbonelli Feb 18, 2024
f13a7ca
skip tests for gwe, swf, prt until mf6 in exes dist supports new mode…
wpbonelli Feb 18, 2024
3d76dad
remove unnecessary --path arg for make-code-json, run black/isort
wpbonelli Feb 18, 2024
d0513d7
name artifacts by OS in test workflow
wpbonelli Feb 18, 2024
564732c
use matrix.os not runner.os (may need to distinguish intel from arm m…
wpbonelli Feb 18, 2024
278de01
use devtools ostags in test workflow artifacts for consistency
wpbonelli Feb 18, 2024
7fdee32
typo
wpbonelli Feb 18, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
Prev Previous commit
Next Next commit
remove unnecessary --path arg for make-code-json, run black/isort
  • Loading branch information
@wpbonelli
wpbonelli committed Feb 18, 2024
commit 3d76dad6eda5c8389e79bed35e513f31139bec95
7 changes: 3 additions & 4 deletions scripts/build_programs.py
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import argparse
import sys
import subprocess
import sys
import textwrap
from pathlib import Path

Expand Down Expand Up @@ -112,9 +112,8 @@ def run_cmd(args) -> bool:
assert run_cmd(
[
"make-code-json",
"-ad", str(path),
"-f",
str(path / "code.json"),
"-ad",
str(path),
"--verbose",
wpbonelli marked this conversation as resolved.
Show resolved Hide resolved
]
), "could not make code.json"
Expand Down