microsoft / microsoft/TypeScript-Sublime-Plugin
Proposal to add a global option for typescript_tsdk
Chưa có ai nhận issue này.
- Ngôn ngữ chính
- JavaScript
- Star
- 1.7k
- Fork
- 236
- Merge trung bình
- 21 ngày 13 giờ
- Pull request đã merge (30 ngày)
- 1
Mô tả
This is somewhat related to issue 499.
Like some other users, I share user settings between machines, and the path to the global TypeScript install is different on each machine. It would be nice if I could set typescript_tsdk: global, or something similar, and the plugin would automatically look for my global install.
I'd be willing to do a pull request. It seems like you could just add another case to the initialize method in editor_client.py. It could look like this:
if tsdk_location == "global":
# Get location of npm packages by running `npm config get prefix`
# See https://stackoverflow.com/questions/4760215/running-shell-command-from-python-and-capturing-the-output
# From there, get path to global TypeScript install.
# Set variables
elif tsdk_location:
# Keep the rest the same
Thoughts?
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 bằng cách đọc phương thức initialize trong typescript/libs/editor_client.py và issue liên quan 499. Kiểm tra cách plugin hiện xử lý typescript_tsdk, sau đó xác minh cách npm config get prefix xác định bản cài đặt TypeScript toàn cục. Hoàn thành khi một thiết lập toàn cục phân giải được bản cài đặt và các đường dẫn tường minh hiện có vẫn tiếp tục hoạt động.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- node.js, python, typescript
- Lĩnh vực
- tooling
- Loại issue
- Tính năng
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 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