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

Add CI to Evaluate Conda ENV setup issues #855

Draft
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

dyt811
Copy link
Member

@dyt811 dyt811 commented Jul 11, 2021

Checklist

GitHub

  • I've given this PR a concise, self-descriptive, and meaningful title
  • I've linked relevant issues in the PR body
  • I've applied the relevant labels to this PR
  • I've assigned a reviewer

PR contents

Description

Currently even though we offer a conda environment.yml file, it has not been undergoing the same rigorous CI validations as the requirements.txt. Sporadic reports of HOURS long resolving conda environment suggests there are potential issues unidentified. The goal of this PR is to add CI integration to allow us to manually validate the performance latest master branch on conda resolution time. It is not meant to be run automatically or at ever PR.

Linked issues

@dyt811 dyt811 added dependencies Pull requests that update a dependency file testing Test related change CI category: TravisCI, GitHub Actions, etc. labels Jul 11, 2021
@dyt811 dyt811 self-assigned this Jul 11, 2021
@dyt811 dyt811 marked this pull request as draft July 11, 2021 17:39
@coveralls
Copy link

coveralls commented Jul 11, 2021

Pull Request Test Coverage Report for Build 1022940133

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 70.295%

Totals Coverage Status
Change from base Build 1020020617: 0%
Covered Lines: 4120
Relevant Lines: 5861

💛 - Coveralls

@dyt811 dyt811 mentioned this pull request Jul 12, 2021
7 tasks
@dyt811
Copy link
Member Author

dyt811 commented Apr 4, 2022

On hold for now as VENV/Virtual env approach are heavily favored in open Canada environment instead.

@dyt811 dyt811 added the on hold Issues/PRs that are on hold because of issues (please explain in the comment) label Apr 4, 2022
@dyt811 dyt811 removed their assignment Apr 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI category: TravisCI, GitHub Actions, etc. dependencies Pull requests that update a dependency file on hold Issues/PRs that are on hold because of issues (please explain in the comment) testing Test related change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants