Skip to content

Execute SET_FUNCTION_ATTRIBUTE before wrapping for #617#620

Merged
MatthieuDartiailh merged 1 commit intonucleic:mainfrom
frmdstryr:update-defaults
Apr 22, 2026
Merged

Execute SET_FUNCTION_ATTRIBUTE before wrapping for #617#620
MatthieuDartiailh merged 1 commit intonucleic:mainfrom
frmdstryr:update-defaults

Conversation

@frmdstryr
Copy link
Copy Markdown
Contributor

@frmdstryr frmdstryr commented Apr 22, 2026

On 3.13+, when a function has defaults it uses the new SET_FUNCTION_ATTRIBUTE instruction. This updates the run_in_dynamicscope to take account for it.

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 22, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 66.06%. Comparing base (ad30b05) to head (f42b078).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #620      +/-   ##
==========================================
+ Coverage   65.99%   66.06%   +0.07%     
==========================================
  Files         270      270              
  Lines       26576    26584       +8     
  Branches     3899     3901       +2     
==========================================
+ Hits        17538    17564      +26     
+ Misses       7990     7975      -15     
+ Partials     1048     1045       -3     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@MatthieuDartiailh
Copy link
Copy Markdown
Member

I will try to think of something but I wonder what is your use case.

@frmdstryr frmdstryr changed the title Draft: Execute SET_FUNCTION_ATTRIBUTE before wrapping for #617 Execute SET_FUNCTION_ATTRIBUTE before wrapping for #617 Apr 22, 2026
@frmdstryr
Copy link
Copy Markdown
Contributor Author

It appears the defaults can already access the dynamicscope so it's not a problem.

@MatthieuDartiailh MatthieuDartiailh merged commit 7895e4c into nucleic:main Apr 22, 2026
15 of 16 checks passed
@frmdstryr frmdstryr deleted the update-defaults branch April 22, 2026 18:55
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.

2 participants