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

Add continuous integration workflow #80

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

TristanCacqueray
Copy link
Contributor

@TristanCacqueray TristanCacqueray commented Oct 14, 2023

This change ensures the library builds and fix some of the doc examples:

$ cargo test --doc 
   Compiling dominator v0.5.34 (/srv/github.com/Pauan/rust-dominator)
    Finished test [unoptimized + debuginfo] target(s) in 0.22s
   Doc-tests dominator

running 14 tests
test src/macros.rs - macros::props (line 682) - compile ... ok
test src/macros.rs - macros::clone (line 583) - compile ... ok
test src/macros.rs - macros::attrs (line 651) - compile ... ok
test src/macros.rs - macros::attrs (line 627) - compile ... ok
test src/fragment.rs - fragment::fragment (line 142) - compile ... ok
test src/macros.rs - macros::attrs (line 638) - compile ... ok
test src/macros.rs - macros::props (line 708) - compile ... ok
test src/macros.rs - macros::props (line 693) - compile ... ok
test src/macros.rs - macros::with_node (line 217) - compile ... ok
test src/macros.rs - macros::styles (line 763) - compile ... ok
test src/macros.rs - macros::styles (line 750) - compile ... ok
test src/macros.rs - macros::styles (line 739) - compile ... ok
test src/macros.rs - macros::apply_methods (line 132) ... ok
test src/fragment.rs - fragment::fragment (line 188) ... ok

test result: ok. 14 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.12s

This change ensures the doc and the tests are passing.
@TristanCacqueray
Copy link
Contributor Author

Hello @Pauan , would it be ok to run rustfmt on add a CI check too?

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.

None yet

1 participant