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.

Sign up for GitHub

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

Jump to bottom

Evm 505: fuzz tests in nightly build #1353

Merged
merged 6 commits into from
Apr 4, 2023
Merged

Evm 505: fuzz tests in nightly build #1353

merged 6 commits into from
Apr 4, 2023

Conversation

Nemanja0x
Copy link
Contributor

@Nemanja0x Nemanja0x commented Apr 1, 2023 •
edited

Description

  1. FuzzTestEVM included into the solution
  2. fuzzAll.sh included into the solution because of the issue
  3. make file updated and fuzz_test.yml created
  4. fuzz tests included in the nightly build

Changes include

  • Bugfix (non-breaking change that solves an issue)
  • Hotfix (change that solves an urgent issue, and requires immediate attention)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (change that is not backwards-compatible and/or changes current functionality)

Checklist

  • I have assigned this PR to myself
  • I have added at least 1 reviewer
  • I have added the relevant labels
  • I have updated the official documentation
  • I have added sufficient documentation in code

Testing

  • I have tested this code with the official test suite
  • I have tested this code manually

@Nemanja0x Nemanja0x self-assigned this Apr 1, 2023
@Nemanja0x Nemanja0x added the feature New update to Polygon Edge label Apr 1, 2023
@Nemanja0x Nemanja0x marked this pull request as ready for review April 1, 2023 16:25
Stefan-Ethernal
Stefan-Ethernal approved these changes Apr 1, 2023
View reviewed changes
Makefile Show resolved Hide resolved
.github/workflows/fuzz-test.yml Show resolved Hide resolved
.github/workflows/nightly.yml Show resolved Hide resolved
vcastellm
vcastellm reviewed Apr 4, 2023
View reviewed changes
fuzzAll.sh Outdated Show resolved Hide resolved
vcastellm
vcastellm approved these changes Apr 4, 2023
View reviewed changes
Copy link
Contributor

@vcastellm vcastellm left a comment

Choose a reason for hiding this comment

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

LGTM aside from missing new lines
Protip: https://editorconfig.org/

.github/workflows/nightly.yml Outdated Show resolved Hide resolved
scripts/fuzzAll Outdated Show resolved Hide resolved
Nemanja0x and others added 2 commits April 4, 2023 17:47
@Nemanja0x @vcastellm
Update .github/workflows/nightly.yml
b950803 
Co-authored-by: Victor Castell <victor@polygon.technology>
@Nemanja0x @vcastellm
Update scripts/fuzzAll
442fea9 
Co-authored-by: Victor Castell <victor@polygon.technology>
@Nemanja0x Nemanja0x merged commit 99166f0 into develop Apr 4, 2023
@Nemanja0x Nemanja0x deleted the EVM-505 branch April 4, 2023 16:19
@github-actions github-actions bot locked and limited conversation to collaborators Apr 4, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@Stefan-Ethernal Stefan-Ethernal Stefan-Ethernal approved these changes

@vcastellm vcastellm vcastellm approved these changes

@goran-ethernal goran-ethernal goran-ethernal approved these changes

@igorcrevar igorcrevar Awaiting requested review from igorcrevar

Assignees

@Nemanja0x Nemanja0x

Labels
feature New update to Polygon Edge
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@Nemanja0x @vcastellm @Stefan-Ethernal @goran-ethernal