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

Implement continous integration for linting and formatting and adress issues #244

Merged
merged 41 commits into from
Aug 26, 2024

Conversation

kedhammar
Copy link

Implemented CI-practices based on repo SciLifeLab/scilifelab_epps and addressed all linting and formatting issues.

separate user-vs-dev requirements and add pytest and codecov to ci

make pkg comparison case-insensitive
apply pytest-cov, try different codecov workflow

try new codecov action

correct syntax
@kedhammar kedhammar self-assigned this Jul 1, 2024
genologics/version.py Outdated Show resolved Hide resolved
genologics/version.py Outdated Show resolved Hide resolved
genologics/version.py Outdated Show resolved Hide resolved
@kedhammar kedhammar requested a review from aanil August 5, 2024 09:57
genologics/lims_utils.py Outdated Show resolved Hide resolved
Co-authored-by: Anandashankar Anil <[email protected]>
Copy link

@aanil aanil left a comment

Choose a reason for hiding this comment

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

👍

genologics/version.py Outdated Show resolved Hide resolved
"requests.Session.get",
return_value=Mock(content=self.reagentkit_xml, status_code=200),
):
assert r.name == "regaentkitname"
Copy link
Member

Choose a reason for hiding this comment

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

Tempting to want to change this, but I don't know where it origins from

Copy link
Author

Choose a reason for hiding this comment

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

afe846e

I just replace regaentwith reagent, 3 hits repo wide.

tests/test_entities.py Outdated Show resolved Hide resolved
Copy link
Member

@alneberg alneberg left a comment

Choose a reason for hiding this comment

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

Looks good! A few tiny comments that you can do what you want with.

@kedhammar kedhammar requested a review from alneberg August 26, 2024 07:42
@kedhammar kedhammar merged commit b93c2a7 into SciLifeLab:master Aug 26, 2024
7 checks passed
@kedhammar kedhammar deleted the ci branch August 26, 2024 07:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants