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

Minor reformatting #4

Merged
merged 2 commits into from
Nov 15, 2023
Merged

Minor reformatting #4

merged 2 commits into from
Nov 15, 2023

Conversation

maanst
Copy link
Owner

@maanst maanst commented Nov 15, 2023

Some minor reformatting suggested by Pycharm.

@maanst maanst merged commit f2d4caa into main Nov 15, 2023
10 checks passed
@maanst maanst deleted the minor_reformatting branch November 15, 2023 13:04
maanst added a commit that referenced this pull request Nov 15, 2023
* Added test timing.

* Removed long-running test for ease of development

* Further clean-up

* Reinstated time-consuming tests

* Removing hardcoding of repo

* Removing reference to saved table. Timing is shown in stdout of each run

* More descriptive name of column

* Updated actions/checkout to v3

* Updated the README, adding curation-only workflow.

* Minor reformatting (#4)

* Pycharm reformatting and removal of redundant parentheses

* Simplifying boolean expressions

---------

Co-authored-by: Marius André Strand <[email protected]>

---------

Co-authored-by: Marius André Strand <[email protected]>

Choose a reason for hiding this comment

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

❤ Very nice! This definitely improves the readability of the code

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.

2 participants