Skip to content

Fix generateTrace with newer TypeScript versions and build: false #730

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

Merged
merged 4 commits into from
Apr 11, 2022

Conversation

mkantor
Copy link
Contributor

@mkantor mkantor commented Apr 9, 2022

Closes #722 and also fixes another issue I noticed along the way: generateTrace did not work unless I was using build: true (I think that was introduced by #691).

Adds tests and updates documentation.

Also update README.md to better reflect the current state of this
feature.
@mkantor mkantor force-pushed the revitalize-generate-trace branch from 66948d7 to 777ee59 Compare April 9, 2022 20:56
Copy link
Collaborator

@piotr-oles piotr-oles left a comment

Choose a reason for hiding this comment

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

Thanks for the PR! I left a few minor comments :)

@mkantor mkantor force-pushed the revitalize-generate-trace branch from 777ee59 to 97a9f40 Compare April 10, 2022 18:27
@mkantor
Copy link
Contributor Author

mkantor commented Apr 10, 2022

@piotr-oles I addressed your feedback; thanks for the quick review. Let me know if you have any other suggestions.

Copy link
Collaborator

@piotr-oles piotr-oles left a comment

Choose a reason for hiding this comment

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

Thanks, looks good!

@piotr-oles piotr-oles merged commit cb327ee into TypeStrong:main Apr 11, 2022
@piotr-oles
Copy link
Collaborator

🎉 This PR is included in version 7.2.4 🎉

The release is available on:

Your semantic-release bot 📦🚀

@mkantor mkantor deleted the revitalize-generate-trace branch April 11, 2022 14:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

generateTrace does not work with newer TypeScript versions
2 participants