Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Test features individually #234

Draft
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

matthiasbeyer
Copy link
Member

Closes #233

This PR aims to resolve #233 - the issue I have right now is that the doctests are written with the json feature, but they fail if I run CI with cargo test --no-default-features --feature ini for example.

If someone knows how to enable the doctests only when json is enabled, that'd be awesome! 😆

Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
…e enabled

Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
…" feature enabled

Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
…th "toml" feature enabled"

This reverts commit de64ae9.
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

unit tests doesn't gate features properly
2 participants