Skip to content
This repository has been archived by the owner on Jul 31, 2023. It is now read-only.

Fix for detecting rubocop output delimiter #520

Closed
wants to merge 1 commit into from

Conversation

tkoenig
Copy link

@tkoenig tkoenig commented Aug 21, 2019

This would be a quick fix for #519. (I tested this quickly modifying the extensions js file)

Given more thought it might be better to get the first instance of FORMATTED_OUTPUT_DELIMITERand not the last one to detect the end of rubocop diagnostics?

Description of change and why it was needed here

  • [] The build passes
  • [] TSLint is mostly happy
  • [] Prettier has been run

This would be a quick fix for rubyide#519. (I tested this quickly modifying the extensions js file)

Given more thought it might be better to get the **first** instance of `FORMATTED_OUTPUT_DELIMITER`and not the last one to detect the end of rubocop diagnostics?
@wingrunr21
Copy link
Collaborator

Hi,

I resolved this in v0.25.0. This PR was still marked as draft and the \n had the potential to not work on Windows. Thanks for the initial bug report!

@wingrunr21 wingrunr21 closed this Sep 2, 2019
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.

2 participants