remove docs support in build.sh, remove test_wheel.sh, other small cleanup

rapidsai / cugraph-gnn

Apache License 2.0

9 stars 15 forks source link

remove docs support in build.sh, remove test_wheel.sh, other small cleanup #60

Closed jameslamb closed 1 month ago

jameslamb commented 1 month ago

Pulling more changes off of #53, to make that diff smaller and get us closer to having full CI running in this repo.

removes docs argument for build.sh (docs building is handled by ci/build_docs.sh)
removes ci/test_wheel.sh
- it looks to me like this is not ever used on #53
removes unnecessary wheel_smoke_test_cugraph.py (copied over from cugraph but not needed here, to be replaced by other scripts you can see in the diff of #53)
other minor cleanup of CI scripts

jameslamb commented 1 month ago

@alexbarghi-nv am I right about this PR (that we want to remove docs-building details in this repo)? If so could you approve? Then I can go get a ci-codeowners / packaging-codeowners approval and we can merge this.

alexbarghi-nv commented 1 month ago

@alexbarghi-nv am I right about this PR (that we want to remove docs-building details in this repo)? If so could you approve? Then I can go get a ci-codeowners / packaging-codeowners approval and we can merge this.

We're moving docs into a new repo and not building them here, so that's correct.

bdice commented 1 month ago

Changes look good, thanks for the clarifying questions above.