question-mark
Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Bug: Linter fails to recognize aliases containing commas

See original GitHub issue

Describe the Bug

This is related to the fix for #509

The following YAML array contains two aliases:

aliases: Scott, "Scott, Jr."

Linter parses this into three aliases.

How to Reproduce

I have attached the data.json file.

See the example above.

Linter produces a multiline array of three aliases:

aliases:
  - Scott
  - '"Scott'
  - 'Jr."'

Expected Behavior

Linter should parse the tags array exactly how Obsidian parses the tags array.

The example should produce a multiline tags array with two tags. The double quotes should be eliminated, and the tag containing the commas should not be escaped with single quotes.

Expected output:

aliases:
  - Scott
  - Scott, Jr.

Device

  • Desktop
  • [?] Mobile

Issue Analytics

  • State:open
  • Created 10 months ago
  • Comments:14

github_iconTop GitHub Comments

1reaction
pjkaufmancommented, Dec 8, 2022

I have not forgotten about this. I just have not had time to sit down and actually determine what it would take to allow for the removing of those parentheses. Hopefully I will have time later this week.

1reaction
pjkaufmancommented, Dec 3, 2022

Gotcha. I will think on this and see if it makes sense and is feasible to remove the quotes for multi-line and other array formats. It just gets a little hard to tell when quotes should be removed without knowing the previous and new formats of the array.

Read more comments on GitHub >

github_iconTop Results From Across the Web

[Bug] tslint doesn't detect webpack aliases? #9227 - GitHub
I have a webpack config with an alias for an import path, VSCode doesn't recognize ... Error messages from tslint are prefixed with...
Read more >
Rules Reference — SQLFluff 1.4.5 documentation
This rule will fail if a single section of whitespace contains both tabs and spaces. ... Reusing table aliases is very likely a...
Read more >
Linters | golangci-lint
Name Description Presets Since asasalint ⚙️ check for pass any as any in variadic func(...any) bugs 1.47.0 bidichk ⚙️ Checks for dangerous unicode character sequences...
Read more >
Why doesn't my Bash script recognize aliases?
commandB , which is an alias to a Bash script. I want to process the same file with these two commands, so I...
Read more >
Why (and How Can I FIx) ESLint import/no-extraneous ...
I solved it simply by setting devDependencies to true. By default it is false. "import/no-extraneous-dependencies": [ "error", { " ...
Read more >

github_iconTop Related Medium Post

No results found

github_iconTop Related StackOverflow Question

No results found

github_iconTroubleshoot Live Code

Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free

github_iconTop Related Reddit Thread

No results found

github_iconTop Related Hackernoon Post

No results found

github_iconTop Related Tweet

No results found

github_iconTop Related Dev.to Post

No results found

github_iconTop Related Hashnode Post

No results found