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 lifecycle hooks, add typescript types, run prettier #18

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

petehunt
Copy link

@petehunt petehunt commented May 8, 2020

hey, thanks for a neat project. i'm messing around with building a hot reloading test runner and i needed some lifecycle hooks to know what was going on. i also added some types and ran it thru prettier.

@petehunt petehunt force-pushed the add-lifecycle-hooks-and-typings branch 2 times, most recently from 1afae1b to 8117511 Compare May 8, 2020 08:08
@sidorares
Copy link
Owner

Thanks Pete! Could you add prettier config you used so that future edits are consistent?

@petehunt petehunt force-pushed the add-lifecycle-hooks-and-typings branch from 8117511 to db762c6 Compare May 8, 2020 08:11
@sidorares
Copy link
Owner

Also if "onReloaded" hook is not part of webpack HMR api we need to add a line to readme ( index.d.ts is also great as documentation, thanks for adding that! )

@petehunt
Copy link
Author

petehunt commented May 9, 2020

will try to get to that this weekend :)

@petehunt
Copy link
Author

@sidorares would you accept a PR to typescript-ify this project?

@sidorares
Copy link
Owner

would you accept a PR to typescript-ify this project?

yes, as long as it's still usable from vanilla js projects.

I also have plans to add proper ejs hooks via vm.Module but never have enough time to do that and whenever I return all my research is outdated

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.

None yet

2 participants