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.

Sign up for GitHub

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

Jump to bottom

Improve code readibility #92

Merged
merged 6 commits into from
Mar 20, 2024
Merged

Improve code readibility #92

merged 6 commits into from
Mar 20, 2024

Conversation

Microwave-WYB
Copy link
Contributor

Changes:

  1. Follow Python PEP8 Style Guide to improve code style
  2. Follow Python PEP484 to type hint variables and interface boundaries (args and returns)
  3. Avoid using f-string or lazy % formatting for log messages to prevent unpredictable behaviors.
  4. Update docstring

@yuema137 yuema137 self-requested a review March 20, 2024 20:31
@yuema137 yuema137 merged commit 423af32 into XENONnT:master Mar 20, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yuema137 yuema137 Awaiting requested review from yuema137

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Microwave-WYB @yuema137