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.

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

Testing and updates #106

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

Testing and updates #106

wants to merge 39 commits into from

Conversation

maxwellflitton
Copy link
Contributor

Thank you for submitting this pull request! We appreciate you spending the time to work on these changes.

What is the motivation?

Type of Change

  • 📚 Examples / docs / tutorials / dependencies update
  • 🔧 Bug fix (non-breaking change which fixes an issue)
  • 🥂 Improvement (non-breaking change which improves an existing feature)
  • 🚀 New feature (non-breaking change which adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to change)
  • 🔐 Security fix

What does this change do?

What is your testing strategy?

Is this related to any issues?

Have you read the Contributing Guidelines?

tsunyoku and others added 30 commits September 18, 2022 14:52
* feat: add pre-commit config

* feat: add dev requirements

* feat: implement sync & async clients

* chore: run formatter

* chore: rename sync client for consistency

* chore: add `__all__` to all files

* chore: refine imports

* feat: improve compatibility for lower py versions

* chore: run formatter

* feat: change `__all__` to use tuples instead of lists

* feat: add extra dunder module attributes

* chore: remove author from setup.py

* chore: remove author dunder attribute and add copyright

* chore: add copyright headers to files

* feat: switch to black

* feat: add license to setup.py

* chore: run formatter

* feat: add more pre-commit hooks

* chore: run formatter

* fix: replace autopep8 with black in dev requirements

* feat: add pyproject.toml file for poetry

* chore: change versioning

* chore: remove unused `requests` dependency

* fix: change `method` type annotation to `str` to support 3.7

* fix: more changes to complete py3.7 support

* feat: add author

* fix: `orjson` in requirements.txt

* feat: move exceptions from folder into single file

* fix: incorrect exception import

* feat: rename `SurrealDBClient` to `SyncSurrealDBClient`

* feat: remove sync client, move & rename async client

* chore: remove un-used requirements file

* force `namespace`, `database`, `username` and `password` to be keyword arguments

* chore: delete setup.py inplace of pyproject.toml

* fix: add `Optional`s on `__aexit__`

* chore: add .vscode to gitignore

* chore: move client.py to clients/http.py

* chore: remove unused dev requirements

* feat: add ws requirements

* feat: add base ws client, increment library version

* feat: add TypedDict for ws request

* feat: add response listening, add remaining request types

* feat: force args after `url` to be keyword

* feat: add username/password auth from ctor

* fix: bugs on authentication

* feat: add ping

* feat: improve function typing

* feat: further improve typing

* fix: remove old client

* Update surrealdb/clients/ws.py

Co-authored-by: Caeden <[email protected]>

* Update surrealdb/clients/ws.py

Co-authored-by: Caeden <[email protected]>

* fix: list type annotation

* feat: remove unused import and add future annotations

* chore: rename `SurrealDBWSClient` to `SurrealDBWebsocketClient`

* chore: remove `SurrealDB` suffix from client names

* fix: port #7 to http client

Co-authored-by: tsunyoku <[email protected]>
Co-authored-by: Caeden <[email protected]>
* fix: replace TypedDict with dataclass for py3.7 compatibility

* fix: response.get("error") -> response.error

* fix: TypedDict notation -> dataclass

* fix: error not being optional

Co-authored-by: tsunyoku <[email protected]>
Co-authored-by: Hitesh <[email protected]>
Co-authored-by: Caeden Perelli-Harris <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.