The Wayback Machine - https://web.archive.org/web/20200614142412/https://github.com/RustPython/RustPython/issues/1714
Skip to content

/ RustPython

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

Jump to bottom

Replace the remaining arg_check! calls with parameter-based arg checking #1714

Open
youknowone opened this issue Jan 29, 2020 · 1 comment · May be fixed by #1953
Open

Replace the remaining arg_check! calls with parameter-based arg checking #1714

youknowone opened this issue Jan 29, 2020 · 1 comment · May be fixed by #1953
Labels
good first issue

Comments

@youknowone
Copy link
Member

@youknowone youknowone commented Jan 29, 2020

Summary

Detailed Explanation

Drawbacks, Rationale, and Alternatives

Unresolved Questions
@coolreader18 coolreader18 changed the title Replace arg_check macro with PyFuncArgs::bind Replace the remaining arg_check! calls with parameter-based arg checking Jan 29, 2020
@coolreader18 coolreader18 removed the RFC label Jan 29, 2020
@coolreader18
Copy link
Contributor

@coolreader18 coolreader18 commented Jan 29, 2020 •
edited

I removed the RFC tag because this is more just some busywork that we've been chipping away at for a while, not necessarily something to decide on. (You can make a normal issue by clicking "Open a blank issue" at the bottom of the issue page, if you didn't know.)
@mrmiywj mrmiywj mentioned this issue Apr 29, 2020
Merged
limeburst added a commit to limeburst/RustPython that referenced this issue Jun 2, 2020
@limeburst
Remove remaining usages of arg_check!
3dd162c 
Close RustPython#1714
@limeburst limeburst linked a pull request that will close this issue Jun 2, 2020
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good first issue
Projects
None yet
Milestone
No milestone
Linked pull requests

Successfully merging a pull request may close this issue.

Remove remaining usages of arg_check!
2 participants
@youknowone @coolreader18
You can’t perform that action at this time.