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 more sphinxJsConfig hooks #139

Merged
merged 1 commit into from
May 3, 2024
Merged

Conversation

hoodmane
Copy link
Member

@hoodmane hoodmane commented May 3, 2024

I added hooks preConvert and postConvert. preConvert receives just the typedoc app and runs right after application bootstrap. (To run earlier I think you'd need to make an actual typedoc plugin). postConvert runs right at the end. I also added a map from typedoc reflections to the corresponding ir item that postConvert can look at. I also added an extra_data dictionary which we pass to Python so that implementations of these hooks can pass extra information to be handled on the Pythons side. (None of the Python-side hooks can use this info yet, it can only be used via a monkeypatch.)

No test coverage for any of this...

I added hooks preConvert and postConvert. preCommit receives just the typedoc
app and runs right after application bootstrap. (To run earlier I think you'd
need to make an actual typedoc plugin). `postConvert` runs right at the end. I
also added a map from typedoc reflections to the corresponding ir item that
postConvert can look at. I also added an extra_data dictionary which we pass to
Python so that implementations of these hooks can pass extra information to be
handled on the Pythons side. (None of the Python-side hooks can use this info
yet, it can only be used via a monkeypatch.)

No test coverage for any of this...
@hoodmane hoodmane force-pushed the to-ir-js-pyodide branch from 32ecd11 to 112ec5d Compare May 3, 2024 14:30
@hoodmane hoodmane merged commit c8fc5a7 into pyodide:master May 3, 2024
6 checks passed
@hoodmane hoodmane deleted the to-ir-js-pyodide branch May 3, 2024 14:35
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.

1 participant