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

create asserts a default page width that update leaves alone

Đã đóng
  • Có gì không ổn?
#184 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ó
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
35/100
Loại issue
Tính năng
Độ rõ ràng
Cần làm rõ
Mức độ hoạt động
Sôi nổi
Công nghệ
go
Lĩnh vực
cli

Hướng nghiên cứu

Start with resolveWidth in cmd/create/create.go and compare its default handling with the update path described in the issue. Read the principles documentation and the export and configuration guidance mentioned here. Done means one width policy is chosen, create and update agree or the exception is explicitly documented, and the relevant behavior is reflected in the project’s guidance.

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

Mô tả

create gives every new page a width of max when the file declares none. resolveWidth (cmd/create/create.go) walks --page-width, then frontmatter page_width, then the project file's page_width, and falls back to pagewidth.DefaultWidth when all three are silent. update does the opposite: an absent page_width makes no width request at all and leaves the live width alone.

That makes the two writing verbs disagree about what an absent field means, and it is the one thing keeping L9 (declared-metadata-is-asserted: a field the file declares is made true of the page; a field it omits leaves the page alone) from holding in both verbs. labels and page_status already hold in both.

The question is a decision rather than a bug fix:

  • Leave Confluence's own default. create makes no width request when nothing declares one, matching update and labels. A new page gets whatever Confluence gives a page created through the API.
  • Keep max as house style, but make it a declaration. For example, export and the docs recommend page_width: max in markfluence.yaml, and create asserts only what is declared.
  • Keep the current behavior and say in the principles doc that create asserts a default width, as an accepted exception.

Whichever it is, update and create should agree, or the difference should be written down on purpose.

Ngôn ngữ chính
Go
Star
4
Fork
1
Merge trung bình
1 giờ 35 phút
Pull request đã merge (30 ngày)
39

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 mozilla/markfluence

Tất cả issue của mozilla/markfluence

Issue tương tự

Thêm issue về Go

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.