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

Depostulate function extensionality #1373

Draft
wants to merge 32 commits into
base: master
Choose a base branch
from

Conversation

fredrik-bakke
Copy link
Collaborator

No description provided.

@fredrik-bakke
Copy link
Collaborator Author

After some preliminary experimentation, there are some technical issues to point out:

  • Named modules with parameters are incompatible with BUILTIN pragmas, so I've refactored all modules that have such a pragma to depend only on files that are not (transitively) dependent on any module that uses funext anywhere.
  • There's an issue with public imports when using named imports. See the current typechecking error. Essentially, such definitions are duplicated in the scope, leading to "ambiguous names"

@fredrik-bakke
Copy link
Collaborator Author

The python scripts will be removed later.

@fredrik-bakke
Copy link
Collaborator Author

fredrik-bakke commented Mar 20, 2025

I've refactored all modules that have such a pragma to depend only on files that are not (transitively) dependent on any module that uses funext anywhere.

*If we get to that stage, I can pull these changes to a different branch for a separate review later.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant