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

Deprecation of fuzz.max_global_rejects config #3153

Closed
2 tasks done
0xYYY opened this issue Sep 9, 2022 · 3 comments · Fixed by #4466
Closed
2 tasks done

Deprecation of fuzz.max_global_rejects config #3153

0xYYY opened this issue Sep 9, 2022 · 3 comments · Fixed by #4466
Assignees
Labels
A-config Area: config A-testing Area: testing C-forge Command: forge T-bug Type: bug

Comments

@0xYYY
Copy link
Contributor

0xYYY commented Sep 9, 2022

Component

Forge

Have you ensured that all of these are up to date?

  • Foundry
  • Foundryup

What version of Foundry are you on?

forge 0.2.0 (f540aa9 2022-09-09T00:05:32.599852Z)

What command(s) is the bug in?

No response

Operating System

macOS (Apple Silicon)

Describe the bug

Deprecate config option max_global_rejects in favor of max_test_rejects under the [fuzz] section.

The deprecation was introduced in #3138 and we should fulfill it in a future release.

Suggest to add this issue to a milestone once the exact version is determined.

@0xYYY 0xYYY added the T-bug Type: bug label Sep 9, 2022
@gakonst gakonst added this to Foundry Sep 9, 2022
@gakonst gakonst moved this to Todo in Foundry Sep 9, 2022
@rkrasiuk rkrasiuk added A-testing Area: testing C-forge Command: forge A-config Area: config labels Sep 9, 2022
@mds1
Copy link
Collaborator

mds1 commented Feb 26, 2023

Since there's currently no versioned releases for breaking changes, we probably should look for the presence of max_global_rejects in config files and error/warn if it's found

@0xYYY
Copy link
Contributor Author

0xYYY commented Mar 1, 2023

Since there's currently no versioned releases for breaking changes, we probably should look for the presence of max_global_rejects in config files and error/warn if it's found

A deprecation warning was already included in this PR #3138. (See the screenshot in the Example section.)

Considering that the warning message has been there for almost half a year now, and that foundry hasn't reach a stable version yet. Would it be acceptable to directly remove fuzz.max_global_rejects in a PR in the near future? If so, i'm happy to take this one.

@mds1
Copy link
Collaborator

mds1 commented Mar 1, 2023

Ah thanks, forgot about that.

Would it be acceptable to directly remove fuzz.max_global_rejects in a PR in the near future? If so, i'm happy to take this one.

Yea I agree it's been long enough to remove that flag now. I'll assign this issue to you, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-config Area: config A-testing Area: testing C-forge Command: forge T-bug Type: bug
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants