This is an automated email from the ASF dual-hosted git repository.
wu-sheng pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/skywalking-eyes.git
The following commit(s) were added to refs/heads/main by this push:
new e910f72 fix: corrrect RoundBracketAsterisk comment style (#272)
e910f72 is described below
commit e910f72bae86171d727742c25f8a4555b7b83893
Author: Bryce Thuilot <[email protected]>
AuthorDate: Tue May 5 03:06:09 2026 -0400
fix: corrrect RoundBracketAsterisk comment style (#272)
Signed-off-by: Bryce Thuilot <[email protected]>
---
assets/styles.yaml | 4 +--
pkg/header/fix_test.go | 8 ++++--
.../testdata/include_test/with_license/testcase.ml | 32 ++++++++++++----------
test/testdata/spdx_content_test/testcase.ml | 4 ++-
4 files changed, 27 insertions(+), 21 deletions(-)
diff --git a/assets/styles.yaml b/assets/styles.yaml
index 869fef4..85dc501 100644
--- a/assets/styles.yaml
+++ b/assets/styles.yaml
@@ -41,8 +41,8 @@
- id: RoundBracketAsterisk
start: '(*'
- middle: '(*'
- end: '(*'
+ middle: ' *'
+ end: ' *)'
- id: CurlyBracketDash
start: '{-'
diff --git a/pkg/header/fix_test.go b/pkg/header/fix_test.go
index d6414b0..0947cdc 100644
--- a/pkg/header/fix_test.go
+++ b/pkg/header/fix_test.go
@@ -84,9 +84,11 @@ func TestRewriteContent(t *testing.T) {
content: `print_string "hello worlds!\n";;
`,
licenseHeader: getLicenseHeader("test.ml", t.Error),
- expectedContent: `(* Apache License 2.0
-(* http://www.apache.org/licenses/LICENSE-2.0
-(* Apache License 2.0
+ expectedContent: `(*
+ * Apache License 2.0
+ * http://www.apache.org/licenses/LICENSE-2.0
+ * Apache License 2.0
+ *)
print_string "hello worlds!\n";;
`},
diff --git a/test/testdata/include_test/with_license/testcase.ml
b/test/testdata/include_test/with_license/testcase.ml
index bbdecc2..2bdd534 100644
--- a/test/testdata/include_test/with_license/testcase.ml
+++ b/test/testdata/include_test/with_license/testcase.ml
@@ -1,18 +1,20 @@
-(* Licensed to the Apache Software Foundation (ASF) under one
-(* or more contributor license agreements. See the NOTICE file
-(* distributed with this work for additional information
-(* regarding copyright ownership. The ASF licenses this file
-(* to you 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.
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements. See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership. The ASF licenses this file
+ * to you 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.
+ *)
let to_string = function Left -> "Left" | Non -> "Non" | Right -> "Right"
diff --git a/test/testdata/spdx_content_test/testcase.ml
b/test/testdata/spdx_content_test/testcase.ml
index 17ae1cc..ad1ecbd 100644
--- a/test/testdata/spdx_content_test/testcase.ml
+++ b/test/testdata/spdx_content_test/testcase.ml
@@ -1,3 +1,5 @@
-(* SPDX-License-Identifier: Apache-2.0
+(*
+ * SPDX-License-Identifier: Apache-2.0
+ *)
let to_string = function Left -> "Left" | Non -> "Non" | Right -> "Right"
