Skip to content

Fix deploy script #423

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Fix deploy script #423

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
5134813
fix comparison
mccalluc May 6, 2025
ecc4a18
update url
mccalluc May 6, 2025
c938b1f
Good catch: forgot to update both readmes
mccalluc May 7, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion README-PYPI.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ Building on what we've learned from [DP Creator](https://github.com/opendp/dpcre
- UI development in Python with [Shiny](https://shiny.posit.co/py/)

You can run DP Wizard locally and upload your own CSV,
or use the [cloud deployment](https://01966942-7eab-da99-0887-a7c483756aa8.share.connect.posit.cloud/) and only provide column names to protect your private data.
or use the [cloud deployment](https://mccalluc-dp-wizard.share.connect.posit.cloud/) and only provide column names to protect your private data.
In either case, you'll be prompted to describe the analysis you need, and then be able to download outputs including:

- A Jupyter notebook which demonstrates how to use [OpenDP](https://docs.opendp.org/).
Expand Down
2 changes: 1 addition & 1 deletion README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ Building on what we've learned from [DP Creator](https://github.com/opendp/dpcre
- UI development in Python with [Shiny](https://shiny.posit.co/py/)

You can run DP Wizard locally and upload your own CSV,
or use the [cloud deployment](https://01966942-7eab-da99-0887-a7c483756aa8.share.connect.posit.cloud/) and only provide column names to protect your private data.
or use the [cloud deployment](https://mccalluc-dp-wizard.share.connect.posit.cloud/) and only provide column names to protect your private data.
In either case, you'll be prompted to describe the analysis you need, and then be able to download outputs including:

- A Jupyter notebook which demonstrates how to use [OpenDP](https://docs.opendp.org/).
Expand Down
2 changes: 1 addition & 1 deletion scripts/deploy.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ echo "Check git..."
git diff --exit-code || die "There should be no local modifications."

BRANCH=`git rev-parse --abbrev-ref HEAD`
[ "$BRANCH" != "main" ] || die "Current branch should be 'main', not '$BRANCH'."
[ "$BRANCH" = "main" ] || die "Current branch should be 'main', not '$BRANCH'."

echo "Check tests..."

Expand Down
2 changes: 1 addition & 1 deletion tests/utils/test_argparse_helpers.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ def test_help():
assert help in readme_pypi_md, "--help content not in README-PYPI.md"

readme_md = (root_path / "README.md").read_text()
assert readme_pypi_md in readme_md, "--help content not in README.md"
assert readme_pypi_md in readme_md, "README-PYPI.md content not in README.md"


def test_arg_validation_no_file():
Expand Down
Loading