Skip to content

Add allow option to no-setup-in-describe - #424

Merged
screendriver merged 1 commit into
mainfrom
feat/no-setup-in-describe-allow-calls
May 22, 2026
Merged

Add allow option to no-setup-in-describe#424
screendriver merged 1 commit into
mainfrom
feat/no-setup-in-describe-allow-calls

Conversation

@lo1tuma

@lo1tuma lo1tuma commented May 20, 2026

Copy link
Copy Markdown
Owner

Summary

  • define setup explicitly as code executed while the suite callback is evaluated
  • add a rule-local allow option for explicitly permitted calls in suite bodies
  • document the stricter default and add regression coverage

Closes #394.

@lo1tuma
lo1tuma requested a review from screendriver May 20, 2026 13:33
@lo1tuma
lo1tuma force-pushed the feat/no-setup-in-describe-allow-calls branch 2 times, most recently from a270191 to f37bbac Compare May 22, 2026 08:32
@screendriver
screendriver force-pushed the feat/no-setup-in-describe-allow-calls branch from f37bbac to 871eb96 Compare May 22, 2026 10:04
@screendriver
screendriver enabled auto-merge May 22, 2026 10:05
@lo1tuma
lo1tuma force-pushed the feat/no-setup-in-describe-allow-calls branch from 871eb96 to 50e719d Compare May 22, 2026 11:45
@screendriver
screendriver merged commit bab72d2 into main May 22, 2026
8 checks passed
@screendriver
screendriver deleted the feat/no-setup-in-describe-allow-calls branch May 22, 2026 11:47
lo1tuma pushed a commit that referenced this pull request Jul 17, 2026
…calls

Add allow option to no-setup-in-describe
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[feature request] no-setup-in-describe: allow allowlisting function names

2 participants