Skip to content

Add 516 compatibility stubs + astype#420

Merged
SpaceManiac merged 8 commits intoSpaceManiac:masterfrom
hry-gh:516-compat
Apr 4, 2025
Merged

Add 516 compatibility stubs + astype#420
SpaceManiac merged 8 commits intoSpaceManiac:masterfrom
hry-gh:516-compat

Conversation

@hry-gh
Copy link
Copy Markdown
Contributor

@hry-gh hry-gh commented Feb 28, 2025

we still need to do for (k, v in list()/alist()) and pragma syntax c but it'll do for now

@LemonInTheDark LemonInTheDark mentioned this pull request Mar 1, 2025
26 tasks
@LemonInTheDark
Copy link
Copy Markdown
Contributor

LemonInTheDark commented Mar 1, 2025

Missing a few other things see linked issue, didn't see this before you're so epic and cool

@hry-gh hry-gh changed the title 516 compat 516 compatibility stubs Mar 14, 2025
@hry-gh hry-gh changed the title 516 compatibility stubs 516 compatibility stubs + astype Mar 14, 2025
@hry-gh hry-gh marked this pull request as ready for review March 14, 2025 14:04
@SpaceManiac
Copy link
Copy Markdown
Owner

SpaceManiac commented Apr 4, 2025

It seems to me that load_ext and astype both syntactically resemble standard proc calls (unlike call_ext which has TWO pairs of parentheses after it instead of one) so I think they don't need their own special Term entries.

Otherwise looks good to me.

@hry-gh
Copy link
Copy Markdown
Contributor Author

hry-gh commented Apr 4, 2025

It seems to me that load_ext and astype both syntactically resemble standard proc calls (unlike call_ext which has TWO pairs of parentheses after it instead of one) so I think they don't need their own special Term entries.

Otherwise looks good to me.

done!

@SpaceManiac SpaceManiac changed the title 516 compatibility stubs + astype Add 516 compatibility stubs + astype Apr 4, 2025
@SpaceManiac SpaceManiac merged commit fcc814d into SpaceManiac:master Apr 4, 2025
1 check passed
@SpaceManiac SpaceManiac added this to the suite v1.10 milestone May 12, 2025
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.

3 participants