Skip to content

Conversation

weswigham
Copy link
Member

@DanielRosenwasser commented in #4915 stating that he didn't like how he lost intellisense for the diagnostics file. Adding this git hook to your repo makes it so that it gets regenerated each time you checkout.

There's a postinstall to run the script to link it into .git/hooks, so a simple npm install (done as part of the existing setup) sets these up, too.

Copy link
Member

Choose a reason for hiding this comment

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

Why not just jake generate-diagnostics?

Copy link
Member Author

Choose a reason for hiding this comment

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

This works when jake isn't installed globally. 😄

Copy link
Member

Choose a reason for hiding this comment

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

💯

@DanielRosenwasser
Copy link
Member

👍

weswigham added a commit that referenced this pull request Sep 25, 2015
@weswigham weswigham merged commit b786687 into microsoft:master Sep 25, 2015
@weswigham weswigham deleted the repo-hooks branch August 17, 2017 23:01
@microsoft microsoft locked and limited conversation to collaborators Jun 19, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants