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

[Merged by Bors] - add simple fuzz test for parser #25

Closed
wants to merge 3 commits into from

Conversation

ozgrakkurt
Copy link
Contributor

@ozgrakkurt ozgrakkurt commented Apr 27, 2023

didn't add it to the ci. can be run by make fuzz-parser but need to install nightly toolchain and cargo-fuzz first.

@ozgrakkurt ozgrakkurt requested a review from sehz April 27, 2023 05:31
@ozgrakkurt ozgrakkurt changed the title add simple fuzz add simple fuzz test for parser Apr 27, 2023
@ozgrakkurt
Copy link
Contributor Author

@sehz I removed Cargo.lock from git since this is a library, is it ok?

@sehz
Copy link
Contributor

sehz commented Apr 27, 2023

Oh. should check in

@ozgrakkurt
Copy link
Contributor Author

Oh. should check in

It says it should be ignored here: rust-lang/cargo#315

@sehz
Copy link
Contributor

sehz commented Apr 27, 2023

Not sure; until now, we always check in cargo.lock and didn't have problems. maybe we need to revisit this. So for now, can keep out

@ozgrakkurt
Copy link
Contributor Author

bors r+

bors bot pushed a commit that referenced this pull request Apr 27, 2023
didn't add it to the ci. can be run by `make fuzz-parser` but need to install nightly toolchain and `cargo-fuzz` first.
@bors
Copy link

bors bot commented Apr 27, 2023

Pull request successfully merged into main.

Build succeeded:

@bors bors bot changed the title add simple fuzz test for parser [Merged by Bors] - add simple fuzz test for parser Apr 27, 2023
@bors bors bot closed this Apr 27, 2023
@bors bors bot deleted the fuzz-test branch April 27, 2023 05:45
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

2 participants