Fix Go doc-comment association for maxPages and getList
@khvn26 がすでに取り組んでいます。
2026年7月28日 から。
評価
この issue はまだ評価されていません。
説明
Summary
Move the maxPages constant and its documentation above the getList doc comment in internal/api/client.go.
Rationale
The current declaration is positioned between the getList documentation and the function. As a result, Go documentation associates the trailing paragraph with maxPages, while getList no longer has a correctly attached doc comment.
Affected area
internal/api/client.gomaxPages(*Client).getList
Required change
Reorder the declarations so that:
maxPagesis immediately preceded by its own// maxPages ...comment.- The
// getList ...comment immediately precedesfunc (c *Client) getList(...).
Acceptance criteria
maxPagesandgetListeach have a correctly attached Go doc comment.- The change is documentation/declaration ordering only; pagination behaviour remains unchanged.
Backlinks
- Deferred from PR #43: https://github.com/Flagsmith/flagsmith-cli/pull/43
- Review comment: https://github.com/Flagsmith/flagsmith-cli/pull/43#discussion_r3666673361
- Requested by: @khvn26
- 主要言語
- Go
- スター
- 12
- フォーク
- 3
- 平均マージ
- 14時間 56分
- マージ済み PR（30日）
- 8
コントリビューションガイド
このリポジトリのコントリビューションガイドは索引されていません
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
Flagsmith/flagsmith-cli のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
Flagsmith/flagsmith-cli#64 ·
-
難易度 3/5 1〜2日 初心者へのやさしさ 68/100
Flagsmith/flagsmith-cli#107 ·
-
難易度 5/5 1週間以上 初心者へのやさしさ 25/100
Flagsmith/flagsmith-cli#95 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 45/100
Flagsmith/flagsmith-cli#93 ·
-
Dependency Dashboard オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 25/100
Flagsmith/flagsmith-cli#90 ·
Flagsmith/flagsmith-cli の issue をすべて見る
似ている issue
-
bug carvel-triage
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
carvel-dev/kapp-controller#1861 ·
-
feature
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
-
難易度 1/5 1時間未満 初心者へのやさしさ 68/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100