mirror of
https://github.com/qmk/qmk_firmware.git
synced 2024-12-05 01:15:19 +00:00
14 KiB
14 KiB
PR チェックリスト
これは、提出された PR を QMK の協力者がレビューする際に何をチェックするのかの非網羅的なチェックリストです。
これらの推奨事項に矛盾がある場合は、このドキュメントに対して issue を開くか、Discord の QMK コラボレータに連絡することをお勧めします。
一般的な PR
- PRは、ソースリポジトリ上の
master
ではないブランチを使って提出する必要があります- これは、あなたの PR にとって別のブランチをターゲットにするという意味ではなく、むしろ自分の master ブランチで作業をしていないという意味です
- もし PR の提出者が自分の
master
ブランチを使っている場合は、マージ後に "git の使い方" ページへのリンクが表示されます - (このドキュメントの最後にはメッセージの内容が含まれます)
- 新しく追加されたディレクトリとファイル名は小文字でなければなりません
- 上流のソースが元々大文字を使っていた場合 (ChibiOS や他のリポジトリからインポートしたファイルなど)、このルールは緩和されるかもしれません
- 十分な正当性がある場合 (既存のコアファイルとの整合性など) は、このルールを緩和することができます。
- ボードデザイナーがキーボードの名前を大文字にした場合は、十分な正当性とはみとめられません
- すべての
*.c
および*.h
ソースファイルの有効なライセンスヘッダ- 一貫性のために GPL2/GPL3 が推奨されています
- 他のライセンスも許可されていますが、GPL と互換性があり、再配布が許可されていなければなりません。異なるライセンスを使うと、PR がマージされるのをほぼ確実に遅らせることになります
- QMK コードベースの「ベストプラクティス」に従う
- これは網羅的なリストではありませんし、時間が経つにつれて修正される可能性が高いです
- ヘッダファイルでは、
#ifndef
インクルードガードの代わりに#pragma once
を使います - 「旧式の」 GPIO/I2C/SPI 関数を使用しない - 正当な理由がない限り、QMK の抽象化を使用しなければなりません (怠惰は正当な理由にはなりません)
- タイミングの抽象化にも従う必要があります:
_delay_ms()
のかわりにwait_ms()
を。(#include <util/delay.h>
も消します)timer_read()
とtimer_read32()
など。 -- タイミング API は timer.h を参照してください
- 新しい抽象化が有用だと思う場合は、次のことをお勧めします:
- 機能が完成するまで自分のキーボードでプロトタイプを作成する
- Discord の QMK コラボレータと話し合う
- 個別のコア変更としてそれをリファクタリングする
- あなたのキーボードからそのコピーを削除する
- PR を開く前にリベースしてマージの競合をすべて修正します (ヘルプやアドバイスが必要な場合は、Discord で QMK コラボレータに連絡してください)。
キーマップの PR
- 特定のボードファイルをインクルードするよりも
#include QMK_KEYBOARD_H
を推奨します - レイヤーは
#define
よりもenum
が好まれます - カスタムキーコードは
#define
ではなくenum
が必要です。最初のエントリには= SAFE_RANGE
が必要です - LAYOUT マクロ呼び出しのパラメータの途中の改行ではバックスラッシュ(
\
)は不要です - スペーシング(コンマまたはキーコードの最初の文字の配置など)に注意を払うと、見栄えの良いキーマップになります
キーボードの PR
終了した PR(インスピレーションを得るために、以前のレビューコメントセットは、自分のレビューのピンポンをなくすのに役立ちます): https://github.com/qmk/qmk_firmware/pulls?q=is%3Apr+is%3Aclosed+label%3Akeyboard
info.json
- 有効な URL
- 有効なメンテナ
- Configurator で正しく表示されること(Ctrl + Shift + I を押してローカルファイルをプレビューし、高速入力をオンにして順序を確認する)
readme.md
- 標準テンプレートがあること
- 書き込みコマンドが
:flash
で終わっていること - 有効なハードウェアの入手方法へのリンク (手配線の場合を除く) -- プライベートな共同購入は問題ありませんが、一回限りのプロトタイプは疑問視されます。オープンソースの場合は、ファイルへのリンクを提供してください
- ボードをブートローダーモードにリセットする方法を明確に説明してください
- キーボードの写真、できれば PCB の写真も添付してください
rules.mk
MIDI_ENABLE
、FAUXCLICKY_ENABLE
、HD44780_ENABLE
は削除されました# Enable Bluetooth with the Adafruit EZ-Key HID
は# Enable Bluetooth
に変更されました- 機能の有効化に関する
(-/+サイズ)
コメントはなくなりました - ブートローダが指定されている場合は、代替ブートローダのリストを削除します
- mcu_selection.mkの同等の MCU と比較した場合、同じ値の場合、デフォルトの MCU パラメータの再定義がないこと
- キーボードの
config.h
PRODUCT
値にMANUFACTURER
を繰り返さないでください#define DESCRIPTION
は要りません- マジックキーオプション、 MIDI オプション、HD44780 コンフィギュレーションは要りません
- ユーザー設定の設定可能な
#define
はキーマップconfig.h
に移動する必要があります - "
DEBOUNCING_DELAY
" の代りに "DEBOUNCE
" を使います - キーボードが QMK で起動するために最低限必要なコードが存在する必要があります
- マトリックスと重要なデバイスの初期化コード
- (カスタムキーコードや特別なアニメーションなど)商用キーボードの既存の機能をミラーリングする場合は、
default
ではないキーマップを使って処理する必要があります
- Vial 関連のファイルまたは変更は QMK ファームウェアで使用されないため受け入れられません (Vial 固有のコアコードは提出またはマージされていません)
keyboard.c
- 空の
xxxx_xxxx_kb()
または他の weak-define のデフォルト実装関数が削除されていること - コメントアウトされた関数も削除されていること
matrix_init_board()
などがkeyboard_pre_init_kb()
に移行されました。keyboard_pre_init* を参照してください- カスタムマトリックスを使用する場合は、
CUSTOM_MATRIX = lite
を選択し、標準のデバウンスを許可します。マトリックスコードの部分置き換え を参照してください - 可能な場合は、独自の
led_update_*()
実装よりも LED インジケータの設定オプションを優先してください。
- 空の
keyboard.h
- 先頭に
#include "quantum.h"
を置きます LAYOUT
マクロは、該当する場合は標準の定義を使用してください- 該当する場合はコミュニティレイアウトマクロ名を使用します (
LAYOUT
/LAYOUT_all
よりも優先されます)
- 該当する場合はコミュニティレイアウトマクロ名を使用します (
- 先頭に
- キーマップの
config.h
- キーボードから
rules.mk
やconfig.h
が重複していないこと
- キーボードから
keymaps/default/keymap.c
QMKBEST
/QMKURL
が削除されていることMO(_LOWER)
およびMO(_RAISE)
キーコードまたは同等のものを使用していて、キーマップに両方のキーを押したときに adjust レイヤーがある場合 - キーマップに直接 adjust レイヤーに入るキーコードがない場合(MO(_ADJUST)
のように)次のように記述します...
...キーマップのlayer_state_t layer_state_set_user(layer_state_t state) { return update_tri_layer_state(state, _LOWER, _RAISE, _ADJUST); }
process_record_user()
内でlayer_on()
、update_tri_layer()
を手動で処理する代わりに。
- default (および via) のキーマップは「素朴」でなければなりません。
- 他のユーザーが独自のユーザー固有のキーマップを開発するための「クリーンな状態」として使用するための最低限のもの。
- これらのキーマップでは標準のレイアウトが推奨されます(可能な場合)
- デフォルトのキーマップは VIA を有効にするべきではありません -- VIA の統合ドキュメント類には
via
という名前のキーマップが必要です。
- PR の提出者は、同じ PR に機能を紹介する個人的な(または豪華な)キーマップを持たせることができますが、「デフォルト」のキーマップに埋め込むべきではありません
- PR の提出者はまた、既存の商用キーボードへ QMK を移植する場合、その商用製品の既存の機能を反映する「製造業者に一致する」キーマップを持つことができます
- PR に VIA の json ファイルを含めないでください。これらは QMK ファームウェアで使われないため QMK リポジトリに属しません -- それらは VIA のキーボードリポジトリに属します。
さらに、ChibiOS に固有で:
- 既存の ChibiOS ボード定義を使用することを強く推奨します。
- 多くの場合、同等の Nucleo ボードは、同じファミリの異なるフラッシュサイズまたはわずかに異なるモデルで使用できます。
- 例:STM32L082KZ の場合、STM32L073RZ に類似しているため、rules.mkで
BOARD = ST_NUCLEO64_L073RZ
を使用できます。
- 例:STM32L082KZ の場合、STM32L073RZ に類似しているため、rules.mkで
- QMK は ChibiOS のアップグレード時のメンテナンス負担が継続的に発生するため、可能な限りカスタムボード定義を持たないように移行しています。
- 多くの場合、同等の Nucleo ボードは、同じファミリの異なるフラッシュサイズまたはわずかに異なるモデルで使用できます。
- ボードの定義が避けられない場合、
board.c
には標準の__early_init()
(通常の ChibiOS ボードの定義と同じ) と空のboardInit()
を実装しなければなりません。- Arm/ChibiOS 早期初期化を参照してください
__early_init()
は、early_hardware_init_pre()
またはearly_hardware_init_post()
で適切に置き換える必要がありますboardInit()
はboard_init()
に移行する必要があります
コアの PR
develop
ブランチをターゲットにする必要があります。これは、その後、breaking change のタイムラインでmaster
にマージされます。- その他の注意事項 TBD
- 投稿された変更の幅を考えると、コアはもっと主観的です
注意事項
人々が自分の master
ブランチを使用する場合、マージ後に以下を投稿します:
For future reference, we recommend against committing to your `master` branch as you've done here, because pull requests from modified `master` branches can make it more difficult to keep your QMK fork updated. It is highly recommended for QMK development – regardless of what is being done or where – to keep your master updated, but **NEVER** commit to it. Instead, do all your changes in a branch (branches are basically free in Git) and issue PRs from your branches when you're developing.
There are instructions on how to keep your fork updated here:
[**Best Practices: Your Fork's Master: Update Often, Commit Never**](https://docs.qmk.fm/#/newbs_git_using_your_master_branch)
[Fixing Your Branch](https://docs.qmk.fm/#/newbs_git_resynchronize_a_branch) will walk you through fixing up your `master` branch moving forward. If you need any help with this just ask.
Thanks for contributing!
レビュープロセス
一般的に、PR がマージの対象となる前に、意味のある(例えば、コードを検査した)2つ(またはそれ以上)の承認を確認したいと考えています。これらのレビューはコラボレータに限られません -- 時間を割いてくれるコミュニティメンバーは誰でも歓迎(奨励)されます。唯一の違いは、チェックマークが緑にならないことですが、それは問題ありません。
また、PR レビューは自由な時間に行われるものです。それは好意で行われるものなので、私たちはレビューに費やす時間に対して、報酬はうけとっていませんし埋め合わせもありません。そのため、私たちがあなたのプルリクエストに取り掛かるのには時間がかかります。家族や生活のことで PR に手が回らなくなることもあり、そして燃え尽き症候群は深刻な懸念です。QMK ファームウェアリポジトリは、毎月平均200件の PR が開かれ、200件の PR がマージされますので、しばらくお待ちください。