Skip to content

Commit 2d8221d

Browse files
Revert "feat: consolidating staging into main and using main going forward as…" (#236)
This reverts commit 9b70f23.
1 parent 9b70f23 commit 2d8221d

55 files changed

Lines changed: 2989 additions & 14914 deletions

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

.devcontainer/devcontainer.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,11 +15,11 @@
1515
"vscode": {
1616
"extensions": [
1717
"hashicorp.terraform",
18-
"charliermarsh.ruff"
18+
"ms-python.black-formatter"
1919
],
2020
"settings": {
2121
"[python]": {
22-
"editor.defaultFormatter": "charliermarsh.ruff",
22+
"editor.defaultFormatter": "ms-python.black-formatter",
2323
"editor.formatOnSave": true
2424
}
2525
}

.gitignore

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -97,9 +97,6 @@ ENV/
9797
env.bak/
9898
venv.bak/
9999

100-
# Local config / dev data
101-
config/local_inst_data.json
102-
103100
# mkdocs documentation
104101
/site
105102

@@ -117,6 +114,3 @@ dmypy.json
117114
# terraform
118115
**/.terraform/*
119116
**/terraform.tfvars
120-
121-
# Cursor rule files
122-
.cursor/

.vscode/launch.json

Lines changed: 0 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -29,21 +29,6 @@
2929
"env": {
3030
"ENV_FILE_PATH": "${workspaceFolder}/src/worker/.env"
3131
}
32-
},
33-
{
34-
"name": "pytest (current file)",
35-
"type": "debugpy",
36-
"request": "launch",
37-
"module": "pytest",
38-
"args": [
39-
"${file}",
40-
"-v",
41-
"-s"
42-
],
43-
"cwd": "${workspaceFolder}",
44-
"env": {
45-
"ENV_FILE_PATH": "${workspaceFolder}/src/webapp/.env"
46-
}
4732
}
4833
],
4934
}

CONTRIBUTING.md

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -10,8 +10,8 @@ To get an overview of the project, please read the [README](README.md) and our [
1010
## Getting started
1111
### Creating Issues
1212

13-
If you spot a problem, [search if an issue already exists](https://github.com/datakind/edvise-api/issues). If a related issue doesn't exist,
14-
you can open a new issue using a relevant [issue form](https://github.com/datakind/edvise-api/issues/new).
13+
If you spot a problem, [search if an issue already exists](https://github.com/datakind/sst-app-api/issues). If a related issue doesn't exist,
14+
you can open a new issue using a relevant [issue form](https://github.com/datakind/sst-app-api/issues/new).
1515

1616
As a general rule, we don’t assign issues to anyone. If you find an issue to work on, you are welcome to open a PR with a fix.
1717

@@ -28,7 +28,7 @@ poetry install --no-interaction
2828
As many other open source projects, we use the famous [gitflow](https://nvie.com/posts/a-successful-git-branching-model/) to manage our branches.
2929

3030
Summary of our git branching model:
31-
- Get all the latest work from the upstream `datakind/edvise-api` repository
31+
- Get all the latest work from the upstream `datakind/sst-app-api` repository
3232
(`git checkout main`)
3333
- Create a new branch off with a descriptive name (for example:
3434
`feature/new-test-macro`, `bugfix/bug-when-uploading-results`). You can
@@ -107,7 +107,7 @@ You can type `pytest` to run your tests, no matter which type of test it is.
107107

108108
## Continuous Integration
109109

110-
We use [GitHub Actions](https://github.com/datakind/edvise-api/actions)
110+
We use [GitHub Actions](https://github.com/datakind/sst-app-api/actions)
111111
for continuous integration.
112112
See [here](https://docs.github.com/en/actions) for GitHub's documentation.
113113

README.md

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -2,14 +2,14 @@
22

33
This repo contains:
44

5-
* [src/webapp/](https://github.com/datakind/edvise-api/tree/develop/src/webapp): The source code for the SST API (which is called by the SST frontend and by any direct API callers)
6-
* [src/worker/](https://github.com/datakind/edvise-api/tree/develop/src/worker): The source code for the SFTP Worker (which calls the SST API)
5+
* [src/webapp/](https://github.com/datakind/sst-app-api/tree/develop/src/webapp): The source code for the SST API (which is called by the SST frontend and by any direct API callers)
6+
* [src/worker/](https://github.com/datakind/sst-app-api/tree/develop/src/worker): The source code for the SFTP Worker (which calls the SST API)
77
* [terraform/]
8-
(https://github.com/datakind/edvise-api/tree/develop/terraform): The Terraform configuration for the SST API/Frontend and other GCP resources including Cloud SQL setup, networking setup, secrets setup
8+
(https://github.com/datakind/sst-app-api/tree/develop/terraform): The Terraform configuration for the SST API/Frontend and other GCP resources including Cloud SQL setup, networking setup, secrets setup
99
* .devcontainer/ and .vscode/: which allow easy setup if you are using VSCode as your IDE.
10-
* [devtools/](https://github.com/datakind/edvise-api/tree/develop/devtools): is a place to put utility scripts
11-
* .github/: contains mostly copied over files when this directory was forked from the student-success-tool repo, so likely much of it is outdated. The only Github action we've added is the [webapp-and-worker-precommit](https://github.com/datakind/edvise-api/blob/develop/.github/workflows/webapp-and-worker-precommit.yml) which is run on every push to develop. This action contains a python linter (we use [black](https://black.readthedocs.io/en/stable/)), and automated runs of the unit tests in the src/webapp/ and src/worker/ directories.
12-
* Additionally, [pyproject.toml](https://github.com/datakind/edvise-api/blob/develop/pyproject.toml) and [uv.lock](https://github.com/datakind/edvise-api/blob/develop/uv.lock) are important for dependency management. At time of writing, the worker is just skeleton code so there's no separate dependency management. In the long-term consider separating out the dependency management for the two programs.
10+
* [devtools/](https://github.com/datakind/sst-app-api/tree/develop/devtools): is a place to put utility scripts
11+
* .github/: contains mostly copied over files when this directory was forked from the student-success-tool repo, so likely much of it is outdated. The only Github action we've added is the [webapp-and-worker-precommit](https://github.com/datakind/sst-app-api/blob/develop/.github/workflows/webapp-and-worker-precommit.yml) which is run on every push to develop. This action contains a python linter (we use [black](https://black.readthedocs.io/en/stable/)), and automated runs of the unit tests in the src/webapp/ and src/worker/ directories.
12+
* Additionally, [pyproject.toml](https://github.com/datakind/sst-app-api/blob/develop/pyproject.toml) and [uv.lock](https://github.com/datakind/sst-app-api/blob/develop/uv.lock) are important for dependency management. At time of writing, the worker is just skeleton code so there's no separate dependency management. In the long-term consider separating out the dependency management for the two programs.
1313

1414

1515
NOTE: this repo was forked from the https://github.com/datakind/student-success-tool repo, which means some of the static files (e.g. CONTRIBUTING.md) may be outdated or may include irrelevant information from that repo. Please update those as you see fit. For information about the specific items listed above, defer to the specific readmes in the relevant directory.

SECURITY.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,4 +8,4 @@ If we verify a reported security vulnerability, our policy is:
88

99
## Reporting a Security Issue
1010

11-
To report any security issues, please [raise an issue](https://github.com/datakind/edvise-api/issues/new/choose) and select **Security issue*
11+
To report any security issues, please [raise an issue](https://github.com/datakind/sst-app-api/issues/new/choose) and select **Security issue*

cloudbuild-webapp.yaml

Lines changed: 0 additions & 55 deletions
This file was deleted.

cloudbuild-worker.yaml

Lines changed: 0 additions & 34 deletions
This file was deleted.

config/local_inst_data.example.json

Lines changed: 0 additions & 60 deletions
This file was deleted.

pyproject.toml

Lines changed: 6 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ dependencies = [
88
"databricks-sdk~=0.38.0",
99
"pydantic~=2.10",
1010
"fastapi[standard]~=0.115.4",
11-
"google-cloud-storage==2.19.0",
11+
"google-cloud-storage~=2.18.2",
1212
"paramiko~=3.5.0",
1313
"cloud-sql-python-connector[pymysql]~=1.14.0",
1414
"sqlalchemy~=2.0.36",
@@ -26,16 +26,13 @@ dependencies = [
2626
"pandas~=2.0",
2727
"six~=1.16.0",
2828
"thefuzz[speedup]~=0.22.1",
29-
"databricks-sql-connector[pyarrow]~=4.2.0",
29+
"databricks-sql-connector~=3.5.0",
3030
"pandera~=0.13",
31-
"mlflow~=2.22",
32-
"cachetools",
33-
"types-cachetools",
34-
"edvise~=0.2.1",
31+
"mlflow~=2.15.0"
3532
]
3633

3734
[project.urls]
38-
Repository = "https://github.com/datakind/edvise-api"
35+
Repository = "https://github.com/datakind/sst-app-api"
3936

4037
[dependency-groups]
4138
dev = [
@@ -53,15 +50,9 @@ dev = [
5350
requires = ["hatchling"]
5451
build-backend = "hatchling.build"
5552

56-
[tool.hatch.metadata]
57-
allow-direct-references = true
58-
5953
[tool.uv]
6054
default-groups = ["dev"]
6155

62-
[tool.uv.sources]
63-
edvise = { git = "https://github.com/datakind/edvise.git", rev = "develop" }
64-
6556
[tool.ruff]
6657
line-length = 88
6758
indent-width = 4
@@ -93,11 +84,8 @@ lines-after-imports = 1
9384
[tool.pytest.ini_options]
9485
minversion = "8.0"
9586
addopts = ["--verbose", "--import-mode=importlib"]
96-
filterwarnings = [
97-
"ignore::DeprecationWarning",
98-
"ignore::FutureWarning:pandera",
99-
]
100-
testpaths = ["src"]
87+
filterwarnings = ["ignore::DeprecationWarning"]
88+
testpaths = ["tests"]
10189

10290
[tool.mypy]
10391
files = ["src"]

0 commit comments

Comments
 (0)