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

Fix conda-package workflow #895

Merged
merged 2 commits into from
Aug 28, 2022
Merged

Conversation

oleksandr-pavlyk
Copy link
Collaborator

conda-package.yml workflow used different orders of channels specified to conda during building and during testing which might be causing action failures experienced in #893 and #894

  • Have you provided a meaningful PR description?

@github-actions
Copy link

@coveralls
Copy link
Collaborator

coveralls commented Aug 27, 2022

Coverage Status

Coverage remained the same at 81.84% when pulling 4764cec on fix-conda-package-worflow into 8414053 on master.

@oleksandr-pavlyk oleksandr-pavlyk force-pushed the fix-conda-package-worflow branch 3 times, most recently from 2eef379 to d033e8c Compare August 28, 2022 12:25
Bump up cache number

Attempt to triage conda-package workflow

source conda shell definition before calling calling activate

test_examples_list must make sure to install the exact version built by build_linux step

Require dpnp>=0.10.1
Import pybind11::error_already_set into dpctl::memory namespace to work-around
compilation error due to use of not-fully qualified identified in
PYBIND11_OBJECT_CVT macro
@oleksandr-pavlyk oleksandr-pavlyk marked this pull request as ready for review August 28, 2022 16:12
@oleksandr-pavlyk oleksandr-pavlyk changed the title Use consistent order of channels during build and test phases Fix conda-package workflow Aug 28, 2022
@oleksandr-pavlyk oleksandr-pavlyk merged commit 5b3a819 into master Aug 28, 2022
@oleksandr-pavlyk oleksandr-pavlyk deleted the fix-conda-package-worflow branch August 28, 2022 16:34
@github-actions
Copy link

Deleted rendered PR docs from intelpython.github.com/dpctl, latest should be updated shortly. 🤞

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