plotly / plotly/plotly.js

[CHORE]: Reintegrate `stackgl_modules` into top level library

Đang mở
#8,016 0 bình luận 1 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

chore P3 plotly-internal size: 1 task
Ngôn ngữ chính
JavaScript
Star
18.3k
Fork
2k
Merge trung bình
2 ngày 12 giờ
Pull request đã merge (30 ngày)
28

Mô tả

Description

Reintegrate the stackgl_modules sub-project into the top level library.

Why should this change be made?

This was originally moved to a subfolder in #6004 to allow for supporting older versions of Node and npm. That support is no longer required and moving this back would allow us to simplify the repo a bit.

Scope
  • Move dependencies into the root package.json
  • Update scripts to handle new location
Notes
  • It's possible that it would still be beneficial to leave this folder as is since it segregates some dev dependencies that most people don't need
  • This was mentioned as a potential option in #7862

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

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

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. 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.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Hướng nghiên cứu

Bắt đầu với root package.json và dự án con stackgl_modules, sau đó xem lại các thay đổi từ #6004 và phương án thay thế được nêu trong #7862. Truy theo các script tham chiếu đến dự án con và chạy các lệnh build và test hiện có. Hoàn tất có nghĩa là các dependency được xử lý từ root và các script hoạt động mà không còn ràng buộc tương thích Node/npm trước đây.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Đánh giá

Công nghệ
javascript, node.js
Lĩnh vực
build-system, tooling
Loại issue
Tái cấu trúc
Độ khó
4/5
Thời gian dự kiến
3-5 ngày
Mức độ hoạt động
Sôi nổi
Độ rõ ràng
Khá rõ ràng
Mức phù hợp với người mới
58/100

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.