influxdata / influxdata/influxdb-client-python
Rename `Point` to `Points`
Chưa có ai nhận issue này.
- Ngôn ngữ chính
- Python
- Star
- 793
- Fork
- 186
- Merge trung bình
- 3 giờ 2 phút
- Pull request đã merge (30 ngày)
- 1
Mô tả
__Proposal:__
According to [Point definition](https://docs.influxdata.com/influxdb/v2.6/reference/key-concepts/data-elements/#point), `Point` has only one field. However, `influxdb-client-python` allows `Point` class to has fields, like so:
https://github.com/influxdata/influxdb-client-python/blob/854fc6c9d844ca4ea801529af5e34c4efaed9df2/tests/test_point.py#L71-L94
So, I think that we should rename `Point` class to `Points` class.
Hướng dẫn đóng góp
Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Hướng nghiên cứu
Bắt đầu với các dòng 71-94 được tham chiếu trong tests/test_point.py và xem lại định nghĩa Point của InfluxDB được liên kết. Tìm kiếm trong client các vị trí sử dụng lớp Point và xác định đầy đủ ảnh hưởng của việc đổi tên lớp này thành Points. Công việc được hoàn tất khi tên lớp công khai và các tham chiếu đến nó được cập nhật nhất quán, đồng thời các bài kiểm thử bị ảnh hưởng đều đạt.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- python
- Lĩnh vực
- api
- Loại issue
- Tính năng
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức độ hoạt động
- Đình trệ
- Độ rõ ràng
- Khá rõ ràng
- Mức phù hợp với người mới
- 35/100