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

[Enhancement][CI] Add Branch Protection, Code Analysis and Project board

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

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

Đánh giá

Độ khó
5/5
Thời gian dự kiến
Hơn một tuần
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ệ
Công nghệ
csharp, github, github-actions
Lĩnh vực
ci-cd, devops, tooling

Hướng nghiên cứu

Trước tiên, hãy xem xét GitHub Action “Build and test” hiện có và khu vực Settings → Code and Automation → Rules → Rulesets. So sánh yêu cầu bảo vệ branch, các kiểm tra Code Scanning và nullability, các release branch cũng như GitHub Project board với các ràng buộc của repository và các quyết định của maintainer. Hoàn tất nghĩa là có một tập hợp các thay đổi repository đã được thống nhất và xác định phạm vi, thay vì thực hiện toàn bộ đề xuất.

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

Mô tả

improvement

Hello @DocSvartz, @stagep, @andrerav,
while reviewing this topic I noticed a few improvements that could help us avoid similar issues in the future. Since I don’t have access to Settings → Code and Automation → Rules → Rulesets, I can’t apply them myself, but I would like to propose the following for your consideration.

1. Branch protection for the default (master) branch

Given that we now have four people who can review pull requests, it would be beneficial to introduce a branch protection rule that:

  • requires all changes to go through a pull request
  • requires at least one approved reviewer before merging into master

This is a common pattern in OSS repositories to ensure code quality and prevent accidental direct commits.

image image

2. Introducing a structured release branch strategy (for backporting)

It might also be helpful to introduce a dedicated release structure like:

  • release
  • release/stable/<version>

This is a common workflow in OSS projects and would allow clean backporting of fixes to already published versions, without forcing users to adopt breaking changes from ongoing development.
This could be valuable for Mapster in the long term, especially regarding stability and predictable package updates.

3. Strengthening CI feedback: code scanning, nullability analysis, and explicit method contracts

In the GitHub Action (“Build and test”), I noticed analyzer annotations such as “possible null reference exception”.
Since unexpected null behavior is undesirable in a published package, I would suggest enabling more comprehensive static analysis and code scanning.

Possible improvements:

  • Enable GitHub Code Scanning (e.g., CodeQL) to detect nullability risks, unsafe patterns, and API-level issues
  • Consider treating selected analyzer warnings as errors or failing CI on critical diagnostics
  • Where applicable, use attributes such as [NotNullIfNotNull], [NotNull], [MaybeNull], etc., to make method contracts explicitly null-aware.

This would allow us to catch potential nullability issues earlier, improve API clarity, and reduce the likelihood of runtime null-related failures for consumers of the package.

→ to implement CI for Code Scanning I would suggest to start with the template provided from GH and maybe someone of you could apply required setup steps that are Mapster specific as there is already a workflow for build & test ?

4. Introducing a GitHub Project for issue tracking and triage

To improve transparency and coordination around ongoing work, it may be beneficial to set up a GitHub Project Board
This would allow us to track, categorize, and triage issues and pull requests more efficiently, while providing a clear overview of priorities, progress, and open tasks.

A project board could help:

  • visualize the current state of issues and PRs
  • support maintainers in triage and planning
  • make contributor workflows more predictable

References you could look into:

It's possible to set up Workflows e.g. along labels that are added so by this we could automate the (base) management for this to not have way more work then it helps us, but we would need to check and improve the available labels then 👍

@andrerav as GitHub preserves the branch protection rules for admin role, I think you will be the one to apply this, if this suggestion would be considerable

Ngôn ngữ chính
C#
Star
5.2k
Fork
411
Merge trung bình
1 ngày 8 giờ
Pull request đã merge (30 ngày)
4

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 MapsterMapper/Mapster

Tất cả issue của MapsterMapper/Mapster

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.