Skip to content

Adds workflow to run tests#12

Merged
ns-rse merged 2 commits intomainfrom
ns-rse/7-ci-tests
Nov 17, 2023
Merged

Adds workflow to run tests#12
ns-rse merged 2 commits intomainfrom
ns-rse/7-ci-tests

Conversation

@ns-rse
Copy link
Copy Markdown
Collaborator

@ns-rse ns-rse commented Nov 14, 2023

Closes #7

Configured to use CodeCov in reporting.

Also ignores Emacs temporary files.

Closes #7

Configured to use CodeCov in reporting.

Also ignores Emacs temporary files.
@ns-rse ns-rse requested a review from SylviaWhittle November 14, 2023 10:14
* `topofileformats/asd.py` imports `matplotlib.pyplot` but `matplotlib` isn't a dependency listed in `pyproject.toml`
* Removes `--mpl` as an option to `pytest` configuration which is in `pyproject.toml` under the
  `[tool.pytest.ini_options]` section.
@codecov-commenter
Copy link
Copy Markdown

Welcome to Codecov 🎉

Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.

Thanks for integrating Codecov - We've got you covered ☂️

Copy link
Copy Markdown
Collaborator

@SylviaWhittle SylviaWhittle left a comment

Choose a reason for hiding this comment

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

Not got the experience to tell if this is correct, but it seems to make sense, thanks @ns-rse !!!

@ns-rse ns-rse merged commit 178c324 into main Nov 17, 2023
@ns-rse ns-rse deleted the ns-rse/7-ci-tests branch November 17, 2023 22:26
@ns-rse ns-rse restored the ns-rse/7-ci-tests branch November 18, 2023 07:34
@ns-rse ns-rse deleted the ns-rse/7-ci-tests branch May 2, 2024 05:36
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.

Automate tests in CI

3 participants