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

[RF] correctly detect undefined variables in rooarglist #18070

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

[RF] correctly detect undefined variables in rooarglist #18070

wants to merge 2 commits into from

Conversation

ferdymercury
Copy link
Contributor

@ferdymercury ferdymercury commented Mar 20, 2025 •
edited
Loading

This Pull request:

Changes or fixes:

Fixes #18013

Checklist:

  • tested changes locally
  • updated the docs (if necessary)

@github-actions GitHub Actions
Copy link

github-actions bot commented Mar 20, 2025 •
edited
Loading

Test Results

    20 files      20 suites   4d 21h 42m 23s ⏱️
 2 738 tests  2 730 ✅   0 💤  8 ❌
52 624 runs  52 438 ✅ 162 💤 24 ❌

For more details on these failures, see this check.

Results for commit 8ea3809.

♻️ This comment has been updated with latest results.

@pcanal pcanal dismissed a stale review March 23, 2025 15:29

Please tell us why you approve this request.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lmoneta lmoneta Awaiting requested review from lmoneta lmoneta is a code owner

@guitargeek guitargeek Awaiting requested review from guitargeek guitargeek is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

@guitargeek guitargeek

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[RF] RooFormula doesn't complain if you use input called x but don't supply it
2 participants
@ferdymercury @guitargeek