Skip to content

Add type-hints in annotation classes #729

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

Merged

Conversation

murtukov
Copy link
Contributor

@murtukov murtukov commented Aug 15, 2020

Add type-hints to annotation classes + common refactoring

@Vincz please check the PR carefully, because I don't know everything, that happens in your code, therefore could make mistakes. I just added type-hints to annotation classes and made general changes in the annotation related code. Ask if you have any questions.

@murtukov murtukov requested a review from Vincz August 15, 2020 02:45
@Vincz
Copy link
Collaborator

Vincz commented Aug 15, 2020

@murtukov
Does this mean that you started looking into annotations ❤️ ? Or it's just that you get sick of my loose comparisons 😸 ?

@murtukov
Copy link
Contributor Author

@Vincz neither of these. I just don't have enough time for complex tasks, so I just go through code here and there and make simple improvements

Copy link
Contributor

@mcg-web mcg-web left a comment

Choose a reason for hiding this comment

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

👏 good job @murtukov. Maybe we can make some minors before merging this one, I don't see nothing blocking it right now

@murtukov murtukov force-pushed the enhancement/add-annotation-type-hints branch from 359ae51 to c13a344 Compare August 16, 2020 00:39
@murtukov murtukov force-pushed the enhancement/add-annotation-type-hints branch from c13a344 to aa8609a Compare August 16, 2020 01:20
@Vincz Vincz mentioned this pull request Aug 16, 2020
@Vincz Vincz self-requested a review August 16, 2020 14:27
@murtukov murtukov merged commit 7295d56 into overblog:master Aug 16, 2020
@murtukov murtukov deleted the enhancement/add-annotation-type-hints branch November 4, 2020 10:23
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