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

Add explicit \use command #1482

Merged
merged 34 commits into from
Jul 28, 2022
Merged

Add explicit \use command #1482

merged 34 commits into from
Jul 28, 2022

Conversation

alerque
Copy link
Member

@alerque alerque commented Jul 19, 2022

Closes #1385
Closes #1486

@alerque alerque added this to the v0.14.0 milestone Jul 19, 2022
@alerque alerque added the enhancement Software improvement or feature request label Jul 19, 2022
@alerque

This comment was marked as outdated.

@alerque alerque changed the title Add explicit \load-package command Add explicit \use command Jul 19, 2022
@alerque
Copy link
Member Author

alerque commented Jul 19, 2022

See discussion on naming this function in relevant #1385.

@alerque alerque marked this pull request as ready for review July 22, 2022 23:25
@alerque alerque requested review from a team and simoncozens as code owners July 22, 2022 23:25
Copy link
Member

@OlivierNicole OlivierNicole left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For the math package, this looks good to me (I only looked at that), since the changes seem pretty systematic and the tests pass.

@alerque alerque mentioned this pull request Jul 26, 2022
alerque added 3 commits July 27, 2022 00:02
Will enable lots of things including passing options to the inputter,
processing files where the content detection might trip up, potentially
ambiguous formats, multiple inputters for the same format, etc.
alerque added 7 commits July 28, 2022 11:53
…with other modules

This is a small step backwards in scope, but it normalizes how all
different module types are instantiated so packages are not a special
case with an extra argument. Realistically we didn't have the class
scope isolated anyway since it was still used in the global, and long
term there are better ways to pass _ENV to functions that we should be
able to use to fix that scope issue. Even if we don't we would want to
change *all* module types, not just one.
@alerque alerque merged commit 10b333f into master Jul 28, 2022
@alerque alerque deleted the package-command branch July 28, 2022 11:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Software improvement or feature request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Inputter options De-conflate script & require
2 participants