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

Make tsc external tool links clickable in WebStorm. #1247

Closed
samreid opened this issue May 17, 2022 · 8 comments
Closed

Make tsc external tool links clickable in WebStorm. #1247

samreid opened this issue May 17, 2022 · 8 comments

Comments

@samreid
Copy link
Member

samreid commented May 17, 2022

As noted in #1173 (comment) it is convenient to trigger tsc from a keystroke in WebStorm and see the results below. It currently looks like this:

image

Note the filename is not hyperlinked like it is for lint:

image

Let's investigate making these links clickable in the IDE, perhaps by using output filters or by figuring out how lint is hyperlinked.

@samreid samreid self-assigned this May 17, 2022
@samreid
Copy link
Member Author

samreid commented May 17, 2022

Same problem noted in https://github.com/microsoft/TypeScript/issues/7238#issuecomment-190436239 and https://youtrack.jetbrains.com/issue/IDEA-48163/External-Tools-no-evident-way-to-set-filter-that-allows-to-navig

samreid added a commit to phetsims/perennial that referenced this issue May 18, 2022
@samreid
Copy link
Member Author

samreid commented May 18, 2022

After the commits, this is working well and I would like to share it with the dev team tomorrow.

@samreid samreid removed their assignment May 18, 2022
samreid added a commit to phetsims/perennial that referenced this issue May 19, 2022
@zepumph zepumph self-assigned this May 19, 2022
@zepumph
Copy link
Member

zepumph commented May 19, 2022

I wonder if we can make this have a default directory of your cwd.

@samreid
Copy link
Member Author

samreid commented May 19, 2022

I wonder if we can make this have a default directory of your cwd.

It sounds like you are describing a use case outside of running in WebStorm, like running from the command line. Can you please clarify?

zepumph added a commit to phetsims/perennial that referenced this issue May 19, 2022
@zepumph
Copy link
Member

zepumph commented May 19, 2022

like running from the command line. Can you please clarify?

Yes, that is what I was saying. I think that this is not that useful though. The purpose of this file is for webstorm. Let's just use your info with no need for updates.

@zepumph zepumph assigned samreid and unassigned zepumph May 19, 2022
@samreid
Copy link
Member Author

samreid commented May 20, 2022

The changes from @zepumph are working well. I noticed an unrelated problem that getting a type error in the sim where the command is run from, doesn't make an absolute path because there are no '../' to detect. I'm seeing output like:

node perennial/js/scripts/absolute-tsc.js mean-share-and-balance/ ../
js/leveling-out/model/LevelingOutModel.ts(73,5): error TS2322: Type 'PhetioGroup<WaterCupModel, [x: string]>' is not assignable to type 'PhetioGroup<WaterCupModel, [x: number]>'.

@samreid
Copy link
Member Author

samreid commented May 20, 2022

Implemented and works great on both my mac and windows. @zepumph can you please test drive it on yours? Close if all is well. Also note the 2nd command line arg no longer necessary.

@samreid samreid assigned zepumph and unassigned samreid May 20, 2022
@zepumph
Copy link
Member

zepumph commented May 20, 2022

Works great! Thanks for the improvement. No other thoughts here.

@zepumph zepumph assigned samreid and unassigned zepumph May 20, 2022
@samreid samreid closed this as completed May 20, 2022
samreid added a commit to phetsims/perennial that referenced this issue Jun 2, 2022
marlitas pushed a commit that referenced this issue Aug 1, 2022
marlitas pushed a commit that referenced this issue Aug 1, 2022
marlitas pushed a commit that referenced this issue Aug 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants