Skip to content

Conversation

@kevinramharak
Copy link
Collaborator

Reimplements #138 but for the monorepo setup.

Changes:

  • Addition of a logger and a LogOutputChannel named Pretty TypeScript Errors
  • A measure method for the logger to measure the execution time of certain code paths
  • changes uriStore to Map instead of a Record.
  • a few early returns and moving of statements beyond those returns to do slightly less work if its not needed.
  • move cache size and supported diagnostic sources to constants
  • move the diagnostics event handler to a separate file

I haven't added the logger.measure calls, because I think we should keep that just for development purposes for now. I'm planning on creating an issue template where we can guide the user to add the cpuprofile and our extension log file. But I need to research how that works first. We can add the measurement statements when that is complete.

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.

2 participants