Skip to content

Commit

Permalink
Merge pull request #8838 from OpenMined/rasswanth/remove-unused-code
Browse files Browse the repository at this point in the history
Remove Unused code paths from the CodeBase
  • Loading branch information
rasswanth-s committed May 22, 2024
2 parents 90dbc56 + 80e450e commit 55036c8
Show file tree
Hide file tree
Showing 69 changed files with 9 additions and 5,145 deletions.
4 changes: 0 additions & 4 deletions .bumpversion.cfg
Original file line number Diff line number Diff line change
Expand Up @@ -44,10 +44,6 @@ first_value = 1

[bumpversion:file:packages/grid/helm/syft/Chart.yaml]

[bumpversion:file:packages/grid/podman/podman-kube/podman-syft-kube.yaml]

[bumpversion:file:packages/grid/podman/podman-kube/podman-syft-kube-config.yaml]

[bumpversion:file:packages/grid/helm/syft/values.yaml]

[bumpversion:file:packages/syftcli/manifest.yml]
2 changes: 1 addition & 1 deletion .github/workflows/cd-syft.yml
Original file line number Diff line number Diff line change
Expand Up @@ -459,7 +459,7 @@ jobs:
author_name: ${{ secrets.OM_BOT_NAME }}
author_email: ${{ secrets.OM_BOT_EMAIL }}
message: "[syft]bump version"
add: "['.bumpversion.cfg', 'VERSION', 'packages/grid/VERSION','packages/syft/PYPI.md', 'packages/grid/devspace.yaml', 'packages/syft/src/syft/VERSION', 'packages/syft/setup.cfg', 'packages/grid/frontend/package.json', 'packages/syft/src/syft/__init__.py', 'packages/grid/helm/syft/Chart.yaml','packages/grid/helm/repo', 'packages/grid/podman/podman-kube/podman-syft-kube.yaml' ,'packages/grid/podman/podman-kube/podman-syft-kube-config.yaml', 'packages/syftcli/manifest.yml', 'packages/syft/src/syft/protocol/protocol_version.json', 'packages/syft/src/syft/protocol/releases/', 'packages/grid/backend/grid/images/worker_cpu.dockerfile','packages/grid/helm/syft/values.yaml','packages/grid/helm/syft']"
add: "['.bumpversion.cfg', 'VERSION', 'packages/grid/VERSION','packages/syft/PYPI.md', 'packages/grid/devspace.yaml', 'packages/syft/src/syft/VERSION', 'packages/syft/setup.cfg', 'packages/grid/frontend/package.json', 'packages/syft/src/syft/__init__.py', 'packages/grid/helm/syft/Chart.yaml','packages/grid/helm/repo', 'packages/syftcli/manifest.yml', 'packages/syft/src/syft/protocol/protocol_version.json', 'packages/syft/src/syft/protocol/releases/', 'packages/grid/backend/grid/images/worker_cpu.dockerfile','packages/grid/helm/syft/values.yaml','packages/grid/helm/syft']"

- name: Changes to commit to Syft Repo during stable release
if: needs.merge-docker-images.outputs.release_tag == 'latest'
Expand Down
50 changes: 0 additions & 50 deletions .github/workflows/rhel-tests.yml

This file was deleted.

78 changes: 0 additions & 78 deletions .github/workflows/vm-tests.yml

This file was deleted.

8 changes: 4 additions & 4 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,14 +3,14 @@ repos:
rev: v4.5.0
hooks:
- id: check-ast
exclude: ^(packages/grid/ansible/|packages/syft/tests/mongomock)
exclude: ^(packages/syft/tests/mongomock)
always_run: true
- id: trailing-whitespace
always_run: true
exclude: ^(docs/|.+\.md|.bumpversion.cfg|packages/syft/tests/mongomock)
- id: check-docstring-first
always_run: true
exclude: ^(packages/grid/ansible/|packages/syft/tests/mongomock)
exclude: ^(packages/syft/tests/mongomock)
- id: check-json
always_run: true
exclude: ^(packages/grid/frontend/|packages/syft/tests/mongomock)
Expand All @@ -25,10 +25,10 @@ repos:
args: ["--assume-in-merge"]
- id: check-executables-have-shebangs
always_run: true
exclude: ^(packages/grid/ansible/|packages/syft/tests/mongomock)
exclude: ^(packages/syft/tests/mongomock)
- id: debug-statements
always_run: true
exclude: ^(packages/grid/ansible/|packages/syft/tests/mongomock)
exclude: ^(packages/syft/tests/mongomock)
- id: name-tests-test
always_run: true
exclude: ^(.*/tests/utils/)|^(.*fixtures.py|packages/syft/tests/mongomock)
Expand Down
6 changes: 3 additions & 3 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ Perform data science on `data` that remains in `someone else's` server

# Quickstart

`Linux``macOS``Windows``Docker``Podman``Kubernetes`
`Linux``macOS``Windows``Docker``Kubernetes`

## Install Client

Expand Down Expand Up @@ -110,7 +110,7 @@ helm install ... --set ingress.class="gce"

## Note:

🚨 Our deployment tool `Hagrid` has been `Deprecated`. For the updated deployment options kindly refer to
🚨 Our old deployment tool `Hagrid` has been `Deprecated`. For the updated deployment options kindly refer to

- 📚 <a href="https://github.com/OpenMined/PySyft/tree/dev/notebooks/tutorials/deployments">Deployments</a>

Expand All @@ -122,7 +122,7 @@ helm install ... --set ingress.class="gce"
# Install Notes

- PySyft 0.8.1 Requires: 🐍 `python 3.10 - 3.12` - Run: `pip install -U syft`
- PyGrid Requires: 🐳 `docker`, 🦦 `podman` or ☸️ `kubernetes`
- PyGrid Requires: 🐳 `docker` or ☸️ `kubernetes`

# Versions

Expand Down
7 changes: 0 additions & 7 deletions docs/source/api_reference/syft.rst
Original file line number Diff line number Diff line change
Expand Up @@ -33,10 +33,3 @@ syft.abstract\_node module
:undoc-members:
:show-inheritance:

syft.gevent\_patch module
-------------------------

.. automodule:: syft.gevent_patch
:members:
:undoc-members:
:show-inheritance:
Loading

0 comments on commit 55036c8

Please sign in to comment.