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

Develop #43

Open
wants to merge 26 commits into
base: develop
Choose a base branch
from
Open

Develop #43

wants to merge 26 commits into from

Conversation

MaKaNu
Copy link

@MaKaNu MaKaNu commented Nov 28, 2023

I recognized the out-of-sync branches develop/master

I've updated the .gitignore to fullfill my needs for virtual environemnts.

I sticked with the changes for py2 support but would really say, this should be dropped completely in future releases (3.0.0?).

I did not changed yet to another build system. I prefer working with poetry, because it provides local development in a logical behavior.

Just one other point: the tox tests in this environment, doesn't finish and I am not very familiar with tox.

FirefighterBlu3 and others added 26 commits September 4, 2021 19:53
Signed-off-by: Aurélien Bompard <[email protected]>
…bc object generation test reliance, pass in app data rather than rely on global or local context scoping.
…declares the docstring for the legacy implementation as well as the py2 workaround for typehinting
Fixes FirefighterBlu3#31 by restoring the boolean return value and redeclares the do…
…e-version-for-2.0.2

update the version to 2.0.2
exclude .venv folder
@MaKaNu
Copy link
Author

MaKaNu commented Nov 28, 2023

The action issue:
It seems to be a problem with src/pam/__internals.py:22:5: F401 'typing.Union' imported but unused

Is it still pep8 CONFORM to support py2? I have some doubts.

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