berthin

Results 7 issues of berthin

Hi, I noticed that `cyaml_load_file` and `cyaml_free` have slightly different interfaces but they could basically use the same type for `data`. You could have: ``` cyaml_err_t cyaml_free( const cyaml_config_t *config,...

Hi, I have noticed that `get_lint` (https://github.com/cheshirekow/cmake_format/blob/master/cmakelang/lint/lint_util.py#L107-L121) function assumes that suppression events need to be sorted: ``` out = [] events = list(self._suppression_events) active_suppressions = set() for record in records:...

Hi, The following cmake command ``` add_custom_target(list-all COMMAND echo cmake build target: ${_all_targets}) ``` triggers the C0113 error as: ``` Missing COMMENT in statement which allows it ``` and I...

Hi, I am running cmake-format and cmake-lint as part of my pre-commit check with the following snippet: ``` # Loop through submodules # cmake-lint: disable=E1120 foreach(IDX RANGE ${_SUBMODULES_COUNT}) ``` cmake-lint...

Hi, using the pre-commit hook `format-npm-groovy-lint`, comments are not formatted correctly. See for instance the following example: ``` class MyClass { /** * Some documentation * * This bla bla...

Hi, I am using the pre-commit hooks and found out that the `format-npm-groovy-lint` messes up the indentation with complex expressions. For example, the following snippet ``` class MyClass { static...