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

Stable Release of Autotype. #62

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

Stable Release of Autotype. #62

wants to merge 16 commits into from

Conversation

Mr-Sunglasses
Copy link
Collaborator

Description

Please describe the changes in your pull request in few words here.

Changes

List the changes done to fix a bug or introducing a new feature.

How to test

Describe the steps required to test the changes proposed in the pull request.

@Mr-Sunglasses
Copy link
Collaborator Author

@tushar5526 Looking for the approval.

Copy link
Owner

@tushar5526 tushar5526 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

docs/Makefile is empty ? Did it get committed accidentally?

"progressbar2",
"colorama",
]
requires-python = ">=3.9"
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ubuntu 20.04 comes with python 3.8 as default. Are there any major reason for deprecating support for python 3.8 ?


- There are two themes in the GUI Script: Dark and Light. By clicking the toggle in the bottom left corner of the window, you can switch between the two.
<img src="https://imgur.com/NjLfWcL.gif)">
pip3 install -r requirements-dev.txt
Copy link
Owner

@tushar5526 tushar5526 Aug 27, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is no requirements-dev.txt or requirements.txt. I see we are also not using pipenv and poetry for package management. What tool are we using here, is it pip?

Put the text inside `code` in `Simulator/simulate_keyboard.py` as follows
pre-commit

pip3 install .
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is better to do an editable install for dev setup.

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pip install -e .

from importlib import resources

try:
import tomllib
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tomllib or tomli not included in pyproject.toml as dependencies. Doing an editable install locally is failing.

@tushar5526
Copy link
Owner

Let's move to poetry as well before making the release. It will probably not take much time (as we have very few dependencies) and ease the dev and deployment process. What are your thoughts @Mr-Sunglasses ?

]
keywords = ["Autotype", "simulator", "commandline"]
dependencies = [
"pynput",
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is better to fix the dependencies' version using ~ or ^ to avoid fails if a dependent dependencies stable version upgrades or does a major release. https://python-poetry.org/docs/dependency-specification/

Copy link
Owner

@tushar5526 tushar5526 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed with some comments

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.

2 participants