Improvement: FindFileOrThrow for dotnet tools caching result between project sessions
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- Mức phù hợp với người mới
- 42/100
Hướng nghiên cứu
Xác định phần triển khai của FindFileOrThrow và kiểm tra cách SessionState hiện đang lưu trữ kết quả. Xác minh rằng cache được giới hạn theo phiên bản Unity, vô hiệu hóa các đường dẫn không tồn tại và tránh tìm kiếm chuyên sâu sau khi mở lại một project; kiểm thử thủ công lần khởi chạy đầu tiên và các phiên project tiếp theo.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Hi, handzlikchris. Thank you for your excellent work.
I noticed that every time project opens, FSR always search deeply into Unity installation path for dotnet tools, which spends seconds to complete. Because of storing in SessionState, first-time project launch can't benefit from it.
In my opinion, utilize EditorPrefs with unityVersion could be better.
Here is my snippet for this:
private static string FindFileOrThrow(string fileName)
{
var cacheKey = $"FSR:FilePath_{fileName}:{Application.unityVersion}";
var result = EditorPrefs.GetString(cacheKey, null);
if (result != null && !File.Exists(result))
{
EditorPrefs.DeleteKey(cacheKey);
result = null;
}
if (result == null)
{
result = Directory
.GetFiles(ApplicationContentsPath, fileName, SearchOption.AllDirectories)
.FirstOrDefault();
if (result == null)
{
throw new Exception($"Unable to find '{fileName}', make sure Editor version supports it. You can also add preprocessor directive 'FastScriptReload_CompileViaMCS' which will use Mono compiler instead");
}
}
EditorPrefs.SetString(cacheKey, result);
return result;
}
- Ngôn ngữ chính
- C#
- Star
- 2.3k
- Fork
- 171
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Chuẩn bị môi trường
Dự án này không cung cấp dev container, Dockerfile hay hướng dẫn đóng góp, nên bạn cần tự thiết lập môi trường: hãy bắt đầu từ README và xem hướng dẫn đóng góp lần đầu của chúng tôi để biết các bước chung.
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.
Issue khác của handzlikchris/FastScriptReload
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 88/100
handzlikchris/FastScriptReload#202 · 2 reaction ·
-
csc.rsp supportĐang mở
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 45/100
-
Failed to find entry-pointsCó thể đã có người làm Có pull request liên kết đang mở hoặc đã được merge. Đang mở
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 38/100
handzlikchris/FastScriptReload#203 · 4 bình luận ·
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 42/100
handzlikchris/FastScriptReload#197 · 1 bình luận ·
Tất cả issue của handzlikchris/FastScriptReload
Issue tương tự
-
documentation
Độ khó 1/5 1-3 giờ Mức phù hợp với người mới 62/100
-
bug misleading tooling trivial
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Docs example projects pinned to a literal library version instead of 3.*Có thể đã có người làm @DaveSkender đã nhận hôm nay. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
facioquo/stock-indicators-dotnet#2372 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
[Docs] officecli-docx SKILL.md still describes `\n` as a soft line break after #262Có thể đã có người làm @giprime đã nhận hôm nay. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
Maintainer thường phản hồi trong vòng 2 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
Facepunch/sbox-public#12097 ·
Maintainer thường phản hồi trong vòng 2 ngày