search

compiler-research / xeus-cpp

Jupyter kernel for the C++ programming language
BSD 3-Clause "New" or "Revised" License
17 stars 24 forks source link

Update testing instructions (test after installation) #106

Closed mcbarton closed 3 months ago

mcbarton commented 3 months ago

This PR updates the instructions so that the tests are performed after installation. @alexander-penev can you merge?

codecov-commenter commented 3 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 57.83%. Comparing base (f2d118d) to head (b487e05).

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/compiler-research/xeus-cpp/pull/106/graphs/tree.svg?width=650&height=150&src=pr&token=9KM610P5A4&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=compiler-research)](https://app.codecov.io/gh/compiler-research/xeus-cpp/pull/106?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=compiler-research) ```diff @@ Coverage Diff @@ ## main #106 +/- ## ======================================= Coverage 57.83% 57.83% ======================================= Files 15 15 Lines 600 600 Branches 59 59 ======================================= Hits 347 347 Misses 253 253 ```
mcbarton commented 3 months ago

@anutosh491 can you merge this PR? It just changes the instructilns so you install within the mamaba environment before testing. The previous instructions were causing confusion due to the tests not being able to find the library if not installed.