Hacktoberfest 2026:メンテナが10月に向けて印を付けた、オープンで初心者向けの issue。 Hacktoberfest の issue を見る

Make e2e tests mandatory for PR merging

オープン
#4,284 コメント 0 件 リアクション 0 件 担当者 1 名 GitHub で見る

メンテナーはふだん 3 日以内に返信

@clairep94 がすでに取り組んでいます。

2026年9月1日 から。

評価

この issue はまだ評価されていません。

説明

Enhancement On Hold
Increasing Access

Unsure

Feature enhancement details

Context:

This issue is a followup from GSOC:

  • @Geethegreat has create an e2e test suite for his GSOC 2026 project
  • We purposely made it so that e2e tests were not required for merging PR's so that the GSOC project did not unnecessarily block other work on the repo
  • We also purposely made it so that e2e tests were gated by approval from @Geethegreat, @clairep94, @khanniie, or @raclim during the GSOC period, so that we did not accidentally overspend on GHA hours

  • Now that GSOC is complete, we should update so that the e2e test suite is mandatory for merging into develop
  • Ideally we should update to:
    • Remove the approval gating
    • Have e2e tests trigger automatically after successful unit tests
    • Add some concurrency rules to ensure this doesn't result in a lot of GHA hours useage
      • Cancel any in-progress runs if a later run is triggered
      • Maybe debounce triggers if this is possible
    • Ideally update to minimize the required amount of time for e2e tests to run:
      • Make app startup in CI faster if possible
      • Caching any installations (some work previously done here already)

--> Will self-assign this issue

Requirements:

  • Access to repo settings > environments & branch protection
主要言語
JavaScript
スター
1.7k
フォーク
1.7k
平均マージ
3日 18時間
マージ済み PR(30日)
7

環境構築

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

processing/p5.js-web-editor のほかの issue

processing/p5.js-web-editor の issue をすべて見る

似ている issue

JavaScript の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。