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

cmd: make :GoRun async #3086

Merged
merged 1 commit into from
Feb 3, 2023
Merged

cmd: make :GoRun async #3086

merged 1 commit into from
Feb 3, 2023

Conversation

bhcleek
Copy link
Collaborator

@bhcleek bhcleek commented Nov 11, 2020

Make go#cmd#Run execute go run asynchronously. This will slightly
depart from the previous handling of stderr output. With the :make
approach, any output to stderr will populate the quickfix window. When
run as a job, though, the output will only be parsed as an error if the
program being run exits non-zero. See
#3085 (comment) for
more information.

Make go#cmd#Run execute `go run` asynchronously. This will slightly
depart from the previous handling of stderr output. With the :make
approach, any output to stderr will populate the quickfix window. When
run as a job, though, the output will only be parsed as an error if the
program being run exits non-zero. See
fatih#3085 (comment) for
more information.
@bhcleek bhcleek added this to the vim-go 1.25 milestone Nov 11, 2020
@bhcleek bhcleek modified the milestones: vim-go 1.25, vim-go 1.26 Apr 17, 2021
@bhcleek bhcleek removed this from the vim-go 1.26 milestone Apr 13, 2022
@bhcleek bhcleek added this to the vim-go v1.29 milestone Feb 2, 2023
@bhcleek bhcleek merged commit bdf157c into fatih:master Feb 3, 2023
@bhcleek bhcleek deleted the run/async branch February 3, 2023 06:58
bhcleek added a commit that referenced this pull request Feb 3, 2023
bhcleek added a commit to bhcleek/vim-go that referenced this pull request Feb 16, 2023
This reverts commit bdf157c, reversing
changes made to 9e64fc1.
bhcleek added a commit to bhcleek/vim-go that referenced this pull request Feb 16, 2023
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.

None yet

1 participant