feat(#108): @ must be at top level#587
feat(#108): @ must be at top level#587Marat-Tim wants to merge 2 commits intoobjectionary:masterfrom
@ must be at top level#587Conversation
🚀 Performance AnalysisAll benchmarks are within the acceptable range. No critical degradation detected (threshold is 100%). Please refer to the detailed report for more information. Click to see the detailed report
|
| # Comment. | ||
| [] > app | ||
| foo > x | ||
| bar > @ |
There was a problem hiding this comment.
@Marat-Tim maybe just this:
[] > app
foo > x
bar > @
This will better illustrate what you expect users to write.
There was a problem hiding this comment.
@yegor256 I don't get it - the example here is Incorrect, but the code you're suggesting doesn't actually violate rules of this lint
|
@h1alexbel please, review this one |
I'm not sure we can rely on the order of objects within a single tag, or that using
@in inner objects will always result in an element withbase='$.@'But if that’s not guaranteed, it seems like implementing this lint may not be possible