Skip to content
This repository was archived by the owner on Jan 22, 2026. It is now read-only.

Fix tests in CI#33

Merged
artob merged 2 commits intoartob:masterfrom
imunproductive:master
Dec 17, 2024
Merged

Fix tests in CI#33
artob merged 2 commits intoartob:masterfrom
imunproductive:master

Conversation

@imunproductive
Copy link
Contributor

Previously due to some weird behavior in cargo test command I implemented a certain workaround that breaks running unit tests (Context).
This PR fixes that and also adds running doctests. I initially wanted to implement it in the same command but its not possible, and doing two commands in a single step doesn't look good in the output, thus I decided to make a separate step for that.

previously only integration tests would run
@imunproductive imunproductive added the bug Something isn't working label Dec 16, 2024
@arto-asimov arto-asimov self-assigned this Dec 16, 2024
@artob artob assigned artob and unassigned arto-asimov Dec 17, 2024
Copy link
Owner

@artob artob left a comment

Choose a reason for hiding this comment

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

LGTM.

@artob artob merged commit 4b38100 into artob:master Dec 17, 2024
3 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants