Skip to content

Allow all line breaks#171

Open
tonystevenj wants to merge 1 commit intomicrosoft:mainfrom
tonystevenj:hitfix/allow-all-line-breaks
Open

Allow all line breaks#171
tonystevenj wants to merge 1 commit intomicrosoft:mainfrom
tonystevenj:hitfix/allow-all-line-breaks

Conversation

@tonystevenj
Copy link

@tonystevenj tonystevenj commented Jun 3, 2024

When pulling our cluster, met a error where some new lines were not correctly parsed. This PR will fix this issue.

Reproduce:
This function with '/r' line ending will cause problem.

.create-or-alter function with (folder="hwtest", docstring="Simple demo function", skipvalidation=true) ['hwtestFunction'] () 
{ // test comment
   event | where timestamp between (start..end) // test comment2
   | take 1
}

image

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.

1 participant