| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
|
|
|
|
|
| |
#### Enable misc-unused-using-decls check ✎
- Enabled the `misc-unused-using-decls` clang-tidy check to enforce removal of unused using-declarations
- Removed several unused using-declarations across multiple files to comply with the new check
- Updated clang-tidy configuration to activate the check instead of suppressing it
<a href="https://nda.ya.ru/t/qa0kX64r7DqvtN"><font size="2">Autodescription by Yandex Code Assistant</font></a>
commit_hash:5dd7bcf766b71f2041902fbf314e13f2b81d9a4d
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
https://clang.llvm.org/extra/clang-tidy/checks/performance/move-const-arg.html
This check useful, as it shows, when a `std::move` is useless, so
to expectations of a programmer are not aligned with reality. The
most useful profit is to see `const T&` arguments, that are
potentially can be replaced with `T`.
Also disabled `CheckTriviallyCopyableMove`,
see https://nda.ya.ru/t/Emvgx5Z67fpZY5 why.
commit_hash:c80c264a9774721aabbac85b350ccdbb8b9a39a2
|
| |
|
|
| |
commit_hash:16513c3f8da40e92c22e497694140f1c1584fb3f
|
| |
|
|
| |
commit_hash:9589937209af8e742c0c6401bd1c232f073856bf
|
| |
|
|
|
| |
https://clang.llvm.org/extra/clang-tidy/checks/modernize/use-designated-initializers.html
commit_hash:d73eb463c7bec2bda4c362aab10af49979a7ddd3
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The check `modernize-use-override` is useful as it:
1. Removes redundant `override` when it is used with a `final`.
2. Removes redundant `virtual` or replace it with a `override`.
So it is more clean for a reader, that overriding happens.
It is enabled not out of order, as it is just an alias to
`cppcoreguidelines-explicit-virtual-functions`.
I also decided to switch a strategy of enabling checks. Now
I will enable only a single rule with a single PR and prefer
rules with a non-breaking autofix.
In the new year with new linter checks! 🎄
commit_hash:e6e233baa90b31e5f65e11837546690c47f71ab5
|
| |
|
|
|
|
|
|
| |
There were patches to fix multiline `STRING_VALUE`s, but `ID_QUOTED`,
was untouched, but actually is able be multiline, so this change fixes
it. Now unclosed `ID_QUOTED` highlightes the whole suffix of a source
until the end or closing `BACKTICK`.
commit_hash:d1262172c798fd4b06cc39980cceb74681d5dd40
|
| |
|
|
|
|
|
|
|
| |
This patch introduces an extension configuration for the Clang Tidy.
It is merged with Arcadia Clang Tidy configuration. To begin with,
more checks for identifiers naming are added.
Documentation: https://nda.ya.ru/t/AhbDZbiF7MKe3M.
commit_hash:3481da4c8df0a4d23a991d4a660ae050d2dc5d33
|
| |
|
|
|
|
|
|
|
|
|
|
| |
Fixed [issues](https://nda.ya.ru/t/e98IQkef7Mhm96):
1. Types had higher precedence than UDF and so `Yson::Parse`
was badly highlighted. So function unit was split to let
types to highlight `Decimal(...)` correctly, as well as
UDFs.
2. There was a hack in a Monaco generator (probably I forgot
to fix it) with hardcoded rules for embedded language, so
JSON rule was forgotted to be deleted.
commit_hash:b24527891525dad65bfbafbc209fd3cb7f087f32
|
| |
|
|
|
|
|
|
|
|
| |
- Highlight embedded languages in Shiki
- Generate Monaco playground template
- Highlight type as type even in function context
- Removed an embedded JSON support
- Fix YQLs highlighting for Monaco and highlight-js
- Fix string embedded language in Monaco
commit_hash:d140266b26c73e9623d878a44e7c0c1152d5c437
|
| |
|
|
|
|
| |
User asked to highlight hints as keywords despite they are actually have
options semantics, because it is more familiar and intuitive.
commit_hash:9c0b63afea781aabfd33189abddb9d0a0ffbaa08
|
| |
|
|
|
|
|
|
| |
The problem was that YQL basic string literals are actually multiline
and so we need to use range pattern for them.
Also this patch highlights embedded JSON as JavaScript in Monarch.
commit_hash:83db61d5f3560ca914fd277df5fef6f242fa0d30
|
| |
|
|
|
|
|
| |
There was a bug with highlighting an identifier a in expression a == b
as an option. So for an option match we expect an exactly = token. So
all hints are enumerated and the solid regex is built.
commit_hash:8df3cd082a7eaa99fc06daa282ba6dd9f0798f6f
|
| |
|
|
| |
commit_hash:a6b640bc576263b62884a0b4187ba79c893e13d9
|
| |
|
|
| |
commit_hash:edf72fd8386c41f858f677152612bd58680506a6
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This patch adds a style check for `yql/essentials/sql/v1`.
To format use `ya style`.
To test use `ya test ya test --test-type clang_format`.
To enable style checking in a module, need to add:
`ENABLE(YQL_STYLE_CPP)`. So in a such way we gradually will
format all YQL modules, expect those, where fluent YQL-builders
are used. Then we will revert activation macro, so switch from
`ENABLE(YQL_STYLE_CPP)` to`ENABLE(YQL_STYLE_CPP_XFAIL)`.
Guide: <https://nda.ya.ru/t/XzkoRIGg7KjXmq>.
commit_hash:fa758a214e094c74821fe896184f30483dd18c55
|
| |
|
|
|
|
|
| |
- Generated Monarch configuration as a JSON.
- Added test files for `*.yql` and `*.yqls`.
- Screenshots: https://nda.ya.ru/t/LgOjOxcs7HJQyS.
commit_hash:dddf747c54780332b54fca3e9e9b52c88a9736f5
|
| |
|
|
|
|
| |
Replaced hardcoded types list at highlighting generator with core
enumerator.
commit_hash:68fa56ed73fabf015a2469ed6bcc4e2481456647
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
- Support `Before` at core `TRegexPattern` and `IGenericLexer`.
- Added `Name` and `Extension` to core `THighlighting`.
- Added `Tighlighting` for `YQLs` factory method.
- Added `--language` option to `yql_highlight`.
- Added `artifact` targets for `YQLs`.
Yes, using the `NSQLTranslation::THighlighting` for
`YQLs` is not correct, but much simplier than generalize
this infrastructure just for a `YQLs`. So here is a
trade-off between development time and a clean code.
Results:
- JetBrains: https://nda.ya.ru/t/PXkZVE8m7H5wHS.
- Vim: https://nda.ya.ru/t/Am-6ZHQa7H5wJi.
- TextMate: https://nda.ya.ru/t/wH0YggAf7H5wKw.
- yql_highlight: https://nda.ya.ru/t/3FaCm57q7H7QSF.
commit_hash:f0e1abb8e7f1b083df531d761b357330bd514cb0
|
| |
|
|
|
|
|
|
| |
- Introduced the Generator interface.
- Made tool to generate content to stdout and file.
- Supported TextMate Bundle, tested on IDEA and TextMate.
- Fixed UDF highlighting priority.
commit_hash:bcbc446a2fe58da3400f0e981a03d821b8f77dae
|
| |
|
|
|
|
|
|
|
| |
- Fixed multiline tokens
- Fixed REAL number display
- Disabled punctuation highlighting
- Refactored by extracting properties
to core highlighting
commit_hash:a2d1eb6e4e49b1cb785b90accbdecebe977faa13
|
| |
|
|
| |
commit_hash:d95cb665e7dd63f67ee4ef7a2602f1c9ec34d26f
|
| |
|
|
|
|
|
|
|
|
| |
Introduced a Vim syntax highlighting for YQL.
This is a replacement for an existing almost
manually written conguration. It uses regexes
generated from the original ANTLR4 grammar.
Now only Default lexer mode is supported.
commit_hash:85fa094593bd9d80373754a492b46ede1a50148d
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Fix `TRegexLexer` performance. Now it is just 2 times slower than a reference ANTLR implementation on Release mode, so merged regexes are 3 times better than scan&compare.

---
- Related to `YQL-19616`
- Related to https://github.com/ydb-platform/ydb/issues/15129
- Related to https://github.com/vityaman/ydb/issues/42
---
Pull Request resolved: https://github.com/ytsaurus/ytsaurus/pull/1278
commit_hash:1529f641172fea13f0d33fbfd06a4827c6efde01
|
|
|
commit_hash:3a624a323006078de71f50747f7b2e8cadba7ccd
|