Skip to content

Commit

Permalink
Sample CI for rust parser
Browse files Browse the repository at this point in the history
  • Loading branch information
lpetre committed Aug 10, 2021
1 parent b30d3d2 commit 1089e28
Showing 1 changed file with 11 additions and 0 deletions.
11 changes: 11 additions & 0 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -86,3 +86,14 @@ jobs:
--sdist
--wheel
--outdir dist/
# build the native parser
native-parser:
name: Native parser
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v2
- uses: actions-rs/toolchain@v1
with:
toolchain: stable
- run: cargo build --manifest-path=native/Cargo.toml --release --all-features

2 comments on commit 1089e28

@zsol
Copy link

@zsol zsol commented on 1089e28 Aug 11, 2021

Choose a reason for hiding this comment

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

Cool! I don't expect you to implement all this, but:

  • we should run cargo test in the workspace too :)
  • I've written some crude benchmarks that we could run with cargo bench and somehow compare with a base revision for PRs.

@zsol
Copy link

@zsol zsol commented on 1089e28 Aug 14, 2021

Choose a reason for hiding this comment

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

I've got this done in zsol@e6ea39d
Thanks for the pointer!

Please sign in to comment.