format will remove the comment by mistake

[tanruixiang]

Comment //! Load balancer will be removed.
Images ID :7a11b1a12b1d

docker run -it --rm -v $(pwd):/github/workspace ghcr.io/korandoru/hawkeye-native format

before format

// Copyright 2022-2023 CeresDB Project Authors. Licensed under Apache-2.0.

//! Load balancer

use macros::define_result;

after format

// Copyright 2023 The CeresDB Authors
//
// Licensed under the Apache License, Version 2.0 (the "License");
// you may not use this file except in compliance with the License.
// You may obtain a copy of the License at
//
//     http://www.apache.org/licenses/LICENSE-2.0
//
// Unless required by applicable law or agreed to in writing, software
// distributed under the License is distributed on an "AS IS" BASIS,
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
// See the License for the specific language governing permissions and
// limitations under the License.

use macros::define_result;

config

headerPath = "Apache-2.0.txt"

includes = [
    "*.rs",
    "*.toml",
    "*.yml"
]

[properties]
inceptionYear = 2023
copyrightOwner = "The CeresDB Authors"

[spencercjh]

Here is an example to demo.

I use the latest code to debug.

https://github.com/korandoru/hawkeye/blob/main/hawkeye-core/src/main/java/io/korandoru/hawkeye/core/header/HeaderParser.java#L115-L127

The rust file's header type is doubleslash_style

The expected behavior should be doing nothing(NOT appended to inPlaceHeader any more) AFTER reading the blank line after // Copyright 2022-2023 CeresDB Project Authors. licensed under Apache-2.0..

@tisonkun I need ur support to fix it. 😿 😿 😿

[tisonkun]

The current algorithm is heuristic. We can try to find a better heuristic algorithm but cannot make a structural correctly one.

I will take a look in this week for thinking about it. Thanks for your report @tanruixiang and investigation @spencercjh .

[tisonkun]

Here is a possible fix - #112