ament / ament/ament_lint

`ament_copyright --add-copyright-year` broken

Đang mở
#414 0 bình luận 1 reaction 0 người được giao Xem trên GitHub
Ngôn ngữ chính
Python
Star
50
Fork
129
Merge trung bình
3 ngày 10 giờ
Pull request đã merge (30 ngày)
3

Mô tả

The `ament_copyright --add-copyright-year` tool is currently broken.

- https://github.com/ament/ament_lint/blob/rolling/ament_copyright/ament_copyright/main.py#L277
this attribute should be named `copyright_identifiers` (with an `s` at the end)
- https://github.com/ament/ament_lint/blob/rolling/ament_copyright/ament_copyright/main.py#L290
the function `search_copyright_information` returns 2 values, not 3

Besides these obvious bugs, I'm also unsure that the `add_copyright_year` function should parse the copyright comments.
the parsed data should already be available in the descriptor objects.

fixing these bugs should be easy.
however, it looks like the tool supports multiple copyrights. How do we deal with them?

Let's say that I have a file with 2 copyrights

```
# Copyright 2021 TheNewCompany

# Copyright 1999 TheOldCompany
```

IMO we should have the tool update only the latest copyright (or maybe the first one found in the document)

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Đánh giá

Issue này chưa được đánh giá.

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.