Skip to content
This repository has been archived by the owner on Mar 21, 2024. It is now read-only.

Enable a disabled test #536

Merged
merged 11 commits into from
Jul 15, 2021
Merged

Enable a disabled test #536

merged 11 commits into from
Jul 15, 2021

Conversation

ant0nsc
Copy link
Contributor

@ant0nsc ant0nsc commented Jul 13, 2021

Undoing #520

Please follow the guidelines for PRs contained here. Checklist:

  • Ensure that your PR is small, and implements one change.
  • Add unit tests for all functions that you introduced or modified.
  • Run PyCharm's code cleanup tools on your Python files.
  • Link the correct GitHub issue for tracking.
  • Update the Changelog file: Describe your change in terms of
    Added/Changed/Removed/... in the "Upcoming" section.
  • When merging your PR, replace the default merge message with a description of your PR,
    and if needed a motivation why that change was required.

@ant0nsc ant0nsc added the no changelog needed CHANGELOG.md does not need to be updated in this PR label Jul 13, 2021
ant0nsc added 3 commits July 13, 2021 12:40
add test coverage for crossval reports
@ant0nsc ant0nsc removed the no changelog needed CHANGELOG.md does not need to be updated in this PR label Jul 14, 2021
@ant0nsc ant0nsc enabled auto-merge (squash) July 14, 2021 14:49
@ant0nsc ant0nsc requested a review from JonathanTripp July 14, 2021 14:49
@ant0nsc ant0nsc disabled auto-merge July 14, 2021 15:47
@ant0nsc ant0nsc enabled auto-merge (squash) July 14, 2021 15:48
JonathanTripp
JonathanTripp previously approved these changes Jul 15, 2021
JonathanTripp
JonathanTripp previously approved these changes Jul 15, 2021
dumbledad
dumbledad previously approved these changes Jul 15, 2021
@ant0nsc ant0nsc dismissed stale reviews from dumbledad and JonathanTripp via 222b932 July 15, 2021 14:45
@ant0nsc ant0nsc disabled auto-merge July 15, 2021 19:53
@ant0nsc ant0nsc merged commit 5399987 into main Jul 15, 2021
@ant0nsc ant0nsc deleted the antonsc/enable_glaucoma branch July 15, 2021 19:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants