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.

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

Figure out where should_skip_current_fnshould go #424

Closed
danielsn opened this issue Aug 12, 2021 · 1 comment · Fixed by #1898
Closed

Figure out where should_skip_current_fnshould go #424

danielsn opened this issue Aug 12, 2021 · 1 comment · Fixed by #1898
Assignees
Labels
[C] Internal Tracks some internal work. I.e.: Users should not be affected.

Comments

@danielsn
Copy link
Contributor

I've moved it to function.rs which is where its used. But that doesn't really satisfy the principle of least surprise.

@tedinski tedinski added the [C] Internal Tracks some internal work. I.e.: Users should not be affected. label Nov 14, 2022
@tedinski
Copy link
Contributor

We should probably try again to eliminate it entirely at this point.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[C] Internal Tracks some internal work. I.e.: Users should not be affected.
Projects
No open projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants