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

Re-enable Linux build#655

Merged
ant0nsc merged 9 commits into
mainfrom
antonsc/linux_build
Feb 4, 2022
Merged

Re-enable Linux build#655
ant0nsc merged 9 commits into
mainfrom
antonsc/linux_build

Conversation

@ant0nsc

@ant0nsc ant0nsc commented Feb 4, 2022

Copy link
Copy Markdown
Contributor

Move pytest to Linux again.
Refactored tests that caused test discovery to slow down (speed up test discovery by 40%)

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 marked this pull request as ready for review February 4, 2022 13:28
@ant0nsc ant0nsc requested a review from mebristo February 4, 2022 13:50
@ant0nsc ant0nsc enabled auto-merge (squash) February 4, 2022 13:54
vmImage: 'windows-2019'
steps:
- template: inner_eye_env.yml
# - job: CreateCondaEnvCache_Windows

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

It would be cleaner to remove this, it's easy to add in if we need it again in the future.

@ant0nsc ant0nsc merged commit d617c81 into main Feb 4, 2022
@ant0nsc ant0nsc deleted the antonsc/linux_build branch February 4, 2022 15:41
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