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

feat: split implementation in two classes #12

Merged
merged 12 commits into from Oct 30, 2021

Conversation

leynier
Copy link
Collaborator

@leynier leynier commented Oct 21, 2021

[BREAKING CHANGES] QvaPayClient was divided into two classes: AsyncQvaPayClient and SyncQvaPayClient. Both classes have the same methods and properties, with the difference that the methods in AsyncQvaPayClient are asynchronous and in SyncQvaPayClient are synchronous.

The _sync folders were generated automatically executing the command unasync qvapay tests.

The code that is added in the _async folders is automatically transformed.

So every time to make a change you must run the command unasync qvapay tests to regenerate the folders _sync with the synchronous version of the implementation.

Improve tests implementation and add pre-commit system to ensure format and style.

The "_sync" folders were generated automatically
executing the command "unasync qvapay tests".

The code that is added in the "_async" folders are
automatically transformed.

So every time to make a change you must run the
command "unasync qvapay tests" to regenerate
the folders "_sync" with the synchronous version
of the implementation.
@leynier leynier added the enhancement New feature or request label Oct 21, 2021
@leynier leynier self-assigned this Oct 21, 2021
@leynier leynier changed the title Split implementation in two classes feature: split implementation in two classes Oct 30, 2021
@leynier leynier changed the title feature: split implementation in two classes feat: split implementation in two classes Oct 30, 2021
@CarlosLugones
Copy link
Contributor

@leynier sorry for the delay, I didn't see any notification related to this. Checking it now.

@CarlosLugones CarlosLugones merged commit 6a53093 into ragnarok22:main Oct 30, 2021
CarlosLugones added a commit that referenced this pull request Oct 20, 2022
feat: split implementation in two classes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants