Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

Integrate updating/checking the config visualizations in the CI

Đang mở
  • Có gì không ổn?
#519 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

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

Đánh giá

Độ khó
4/5
Thời gian dự kiến
3-5 ngày
Mức phù hợp với người mới
25/100
Loại issue
Tính năng
Độ rõ ràng
Cần làm rõ
Mức độ hoạt động
Đình trệ
Lĩnh vực
ci-cd, tooling

Hướng nghiên cứu

Xem xét các script trực quan hóa và kiểm tra từ #514, workflow CI và các phần bổ sung cho hook pre-commit được tham chiếu trong các commit fd9a639 và 1e0aea5. Xác định cách làm cho các tệp DOT và PNG được tạo ra có tính tái lập, sau đó khôi phục các bước kiểm tra trong CI và pre-commit để phát hiện các thay đổi mà không tạo ra cảnh báo giả.

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

Mô tả

In #514, I added config visualizations and a script that generates them. I had also added a script that checks if the DOT files were updated after running the visualization script again and which was triggered in the CI.

I had also added the update and check in the pre-commit hook, but since the precice-config-visualizer at that time required additional dependencies for the GUI, which was not possible to disable at that time, decided to not add this to the pre-commit in the end.

After getting an easy-to-install variant of the precice-config-visualizer, we can add this back into the pre-commit.

We I also disagreed with @fsimonis on how to implement the checks in the workflow. While I am also not happy having the automatically generated DOT files in the repository, and while we could generate them in the website CI, I would still like to keep the PNG files in the tutorials repository, because then the README files remain readable offline and on GitHub. The question would be if we can make them reproducible, so that we don't get false alerts to update them.

Commit that removed the checking and CI part: https://github.com/precice/tutorials/pull/514/commits/fd9a639849764ad76932613bf6ce5a88b11f3798

Commit that removed the pre-commit additions: https://github.com/precice/tutorials/pull/514/commits/1e0aea57438b208a13cbb5c94cc656017cfa3eef

Ngôn ngữ chính
C
Star
143
Fork
169
Merge trung bình
1 ngày 17 giờ
Pull request đã merge (30 ngày)
17

Chuẩn bị môi trường

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.

Issue khác của precice/tutorials

Tất cả issue của precice/tutorials

Issue tương tự

Thêm issue về C

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.