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

Feed correct line number to gvim on launch #5202

Merged
merged 1 commit into from
Oct 1, 2018
Merged

Conversation

identor
Copy link
Contributor

@identor identor commented Oct 1, 2018

Clicking Error overlay should feed the correct line number for gvim.

Relates to #2636.

@Timer
Copy link
Contributor

Timer commented Oct 1, 2018

Does it need added to COMMON_EDITORS_LINUX and potentially isTerminalEditor?

@Timer Timer modified the milestones: 2.0.x, 2.x Oct 1, 2018
@identor
Copy link
Contributor Author

identor commented Oct 1, 2018

Hi @Timer ,

Have tested this, successfully able to go to the line number without adding it to COMMON_EDITORS_LINUX.

But might as well add it since I'm referencing: #2636

@Timer
Copy link
Contributor

Timer commented Oct 1, 2018

It should be able to launch without you specifying it via the EDITOR variable. Can you test this case please?

@identor
Copy link
Contributor Author

identor commented Oct 1, 2018

Testing launch without specifying EDITOR variable. I'll update this pull request once done.

Clicking Error overlay should feed the correct line number for gvim.

Added `gvim` in `COMMON_EDITORS_LINUX`, launches new gvim when it is running.

Tested with gvim 8.0.707 on a Debian 4.9.110-3+deb9u4 (2018-08-21) x86_64 GNU/Linux VM

Relates to facebook#2636.
@identor
Copy link
Contributor Author

identor commented Oct 1, 2018

Added gvim in COMMON_EDITORS_LINUX, launches new gvim when it is running.

Tested with gvim 8.0.707 on a Debian 4.9.110-3+deb9u4 (2018-08-21) x86_64 GNU/Linux VM

@gaearon gaearon merged commit bf5a4bf into facebook:master Oct 1, 2018
@gaearon
Copy link
Contributor

gaearon commented Oct 1, 2018

Thanks

@gaearon gaearon modified the milestones: 2.x, 2.0 Oct 1, 2018
@lock lock bot locked and limited conversation to collaborators Jan 19, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants