Skip to content

Type hints #38

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
merged 7 commits into from
Apr 24, 2025
Merged

Type hints #38

merged 7 commits into from
Apr 24, 2025

Conversation

rozyczko
Copy link
Member

@rozyczko rozyczko commented Apr 23, 2025

Added type hints throughout the code. Additionally, doctrings were added to methods and classes.

The branch is based off of unit_tests, so I could immediately check the code for issues.
This is why it should be merged into that branch first, before unit_tests is merged with develop

@rozyczko rozyczko requested a review from AndrewSazonov April 23, 2025 07:24
@rozyczko rozyczko added the [scope] maintenance Code/tooling cleanup, no feature or bugfix (major.minor.PATCH) label Apr 23, 2025
@rozyczko rozyczko added this to the New API Stabilization milestone Apr 23, 2025
@rozyczko rozyczko moved this to In Review in New EasyDiffractionLib Apr 23, 2025
@rozyczko rozyczko changed the base branch from master to unit_tests April 23, 2025 07:29
Copy link
Member

@AndrewSazonov AndrewSazonov left a comment

Choose a reason for hiding this comment

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

Overall, it looks good. There were quite a lot of changes, so it's a bit tricky to go through everything in detail 🙂

One minor thing I noticed - it wasn't clear to me why Any was used in some places instead of more specific types like SampleModels, Experiments, Calculator, etc.

@rozyczko rozyczko merged commit c624e65 into unit_tests Apr 24, 2025
8 checks passed
@github-project-automation github-project-automation bot moved this from In Review to Done in New EasyDiffractionLib Apr 24, 2025
@rozyczko rozyczko deleted the type_hints branch May 2, 2025 05:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[scope] maintenance Code/tooling cleanup, no feature or bugfix (major.minor.PATCH)
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants