Skip to content

[WIP] Test code suggestions for PR5#6

Closed
jsnel wants to merge 7 commits intoglotaran:mainfrom
jsnel:pr5_improvements
Closed

[WIP] Test code suggestions for PR5#6
jsnel wants to merge 7 commits intoglotaran:mainfrom
jsnel:pr5_improvements

Conversation

@jsnel
Copy link
Copy Markdown
Member

@jsnel jsnel commented Jul 27, 2024

This is a draft version of PR #5 to test some code suggestions in the CI.

NOTE: this DRAFT PR is not for merging in (hence marked as DRAFT), code should be merged in via @anmolbhatia05 's PR

Anmol and others added 7 commits July 15, 2024 18:43
- Refactored several functions to improve readability and maintainability
- Replaced os module with pathlib for file operations
- Improved docstrings for clarity and consistency and satify linters
- Renamed 'widget' to '_widget' in widget.py to facilitate unit testing
- Added a new test case for the _simulate function in widget.py
-
@codecov-commenter
Copy link
Copy Markdown

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

@jsnel
Copy link
Copy Markdown
Member Author

jsnel commented Jul 27, 2024

Test succesfull, closing this test draft PR again.

@jsnel jsnel closed this Jul 27, 2024
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