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

Add typings and development setup #48

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

LindezaGrey
Copy link

Hey there,

this pull request changes the following things:

  • remove python2 support
  • add typing
  • add development setup

In order to make it easier to contribute i added a devcontainer setup, which can be used with Github Codespaces in order to directly run and test the code in the browser.

devcontainer

I updated the code with all necessary typing, to address this issue #47
Additionally it was checked by ruff & mypy and formatted with black. All tests still pass.

Create Dockerfile

Create docker-compose.yml

Update docker-compose.yml

add more config

remove python2 support
add typing
add development setup

merge commits
@davidaurelio
Copy link
Owner

hey @LindezaGrey, sorry for not reacting earlier. I will get to this over the weekend. Thanks for contributing!

@JasperSui
Copy link

Hi @davidaurelio, just wonder when will 1.4.0 be released, do you have any concerns about this PR?

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.

3 participants