Skip to content
This repository was archived by the owner on Jan 16, 2025. It is now read-only.

Imports in __init__.py #8

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

Conversation

IvanKirpichnikov
Copy link
Owner

No description provided.

@@ -0,0 +1,4 @@
from .client import KeycloakClient
from .factory import KeycloakClientFactory
from .errors import KeycloakError, UnauthorizedError, UserExistsError
Copy link
Owner Author

@IvanKirpichnikov IvanKirpichnikov Oct 24, 2024

Choose a reason for hiding this comment

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

py from .errors import KeycloakError, UnauthorizedError, UserExistsError
It's unnecessary

Copy link
Owner Author

Choose a reason for hiding this comment

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

add __all__

Copy link

@hntuan7684 hntuan7684 Oct 25, 2024

Choose a reason for hiding this comment

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

I has modify by your request on branch develop 483fa3f

Copy link
Owner Author

Choose a reason for hiding this comment

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

There is no need for imports here

@@ -0,0 +1,24 @@
import access_token
Copy link
Owner Author

Choose a reason for hiding this comment

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

What kind of import?

Choose a reason for hiding this comment

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

I has modify kind of import access_token 3eea470

@IvanKirpichnikov IvanKirpichnikov changed the base branch from develop to master October 24, 2024 16:19
@IvanKirpichnikov IvanKirpichnikov changed the base branch from master to develop October 24, 2024 16:19
__all__ = [
'KeycloakClient',
'KeycloakClientFactory',
]

Choose a reason for hiding this comment

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

I has modify __all__ and delete errors

@@ -0,0 +1 @@

Choose a reason for hiding this comment

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

I has changed

AccessTokenCertConf,
AccessTokenAccess,
AccessToken,
)

Choose a reason for hiding this comment

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

I has modify kind of import access_token

@IvanKirpichnikov
Copy link
Owner Author

fix ruff and mypy, please

@IvanKirpichnikov
Copy link
Owner Author

write in console ruff check src --fix please

@IvanKirpichnikov
Copy link
Owner Author

merge your branch with my develop. I added a config for ruff

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants