GitHub Pages publishes the whole repo: dev files and stray Jekyll pages
まだ誰も着手していません。
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 72/100
- issue の種類
- バグ
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
- 技術スタック
- github
- 領域
- devops
調査の方向性
問題は、GitHub Pages のビルドが追跡対象のすべてのファイルを公開してしまうことです。除外する前に、リポジトリのルート(_config.yml が置かれる場所)から始め、ゲームに必要なファイルの一覧を sw.js で確認してください。README.md の Deployment セクションを読み、配信対象のページが index.html だけであることを確認してください。完了条件は、受け入れ基準のパス(package.json、tests/unit/ai.test.js、README.md、CHANGELOG.html)がライブサイトで 404 を返すこと、ゲームと sw.js 内のすべてのファイルが引き続き 200 を返すこと、そしてゲームがオフラインでも動作し続けることです。
索引モデルが issue の本文から書いたものです。
説明
Summary
GitHub Pages builds the site from the root of main with Jekyll, so every tracked file is published, not only the game. Dev files are served, and each Markdown file becomes an unlinked, GitHub-themed page titled "Static_Chess" that search engines can index next to the game.
Where
- Pages settings: source
main/(root), legacy (Jekyll) build - No
_config.ymland no.nojekyllin the repository root index.html:1(the only page that is meant to be the site)
Details
Checked on the live site at 0144de7. All of these return 200:
| URL path | What it is |
|---|---|
/package.json, /package-lock.json (203 KB), /eslint.config.mjs |
dev tooling |
/tests/unit/ai.test.js, /tests/helpers/load-game.js (and the rest of tests/) |
test suite |
/README.md, /CHANGELOG.md, /docs/ARCHITECTURE.md |
raw Markdown |
/CHANGELOG.html, /docs/ARCHITECTURE.html, /docs/ROADMAP.html, /docs/RELEASE.html, /releases/, /releases/RELEASE_NOTES_v0.1.0.html, /releases/RELEASE_NOTES_v0.2.0.html, /releases/RELEASE_GUIDE_v0.1.0.html |
Jekyll-rendered pages |
The rendered pages use GitHub's default theme, not the game's styles. Their heading is the repository name Static_Chess, their <title> ends in | Static_Chess, and each one requests /Static_Chess/favicon.ico, which is a 404. Nothing on the game page links to them.
Nothing sensitive is exposed: the repository is public and these files are the same ones visible on GitHub.
Impact
- Search engines can index eight off-brand pages and the raw Markdown as part of the site.
- Every push to
mainruns a Jekyll build over files that are not part of the game. - Anything added to the repository later is published automatically, whether or not it was meant to be.
Steps to reproduce
- Open
https://tmhsdigital.github.io/Static_Chess/CHANGELOG.html. - Open
https://tmhsdigital.github.io/Static_Chess/package-lock.json.
Suggested fix
Smallest change: add a _config.yml at the repository root that excludes everything that is not the game, for example:
exclude:
- README.md
- CHANGELOG.md
- docs
- releases
- tests
- package.json
- package-lock.json
- eslint.config.mjs
Alternative: deploy with a GitHub Actions workflow that uploads only index.html, css/, js/, assets/, manifest.webmanifest and sw.js, and switch the Pages source to "GitHub Actions". This is more work and needs a settings change by a maintainer, but it publishes an explicit list instead of an exclusion list.
Acceptance criteria
-
/package.json,/package-lock.json,/tests/unit/ai.test.js,/README.mdand/CHANGELOG.htmlreturn 404 on the live site - The game at
/and every file listed insw.jsstill return 200 - The game still installs and opens offline
Getting started
- Relevant files:
index.html,sw.js(the list of files the game needs),README.md(Deployment section) - How to run tests:
npm test,npm run lint,npm run format - Related docs: GitHub Pages documentation on configuring Jekyll and on custom workflows
Filed by repo-audit/pages-audit at 0144de7. Severity: low. Effort: S. audit-key: enhancement|index.html|pages-published-files
- 主要言語
- JavaScript
- スター
- 4
- フォーク
- 2
- 平均マージ
- 4時間 8分
- マージ済み PR(30日)
- 1
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
TMHSDigital/Static_Chess のほかの issue
-
documentation good first issue severity: low
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
TMHSDigital/Static_Chess#79 ·
-
enhancement good first issue severity: low
難易度 1/5 1時間未満 初心者へのやさしさ 88/100
TMHSDigital/Static_Chess#77 ·
-
enhancement good first issue severity: low
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
TMHSDigital/Static_Chess#76 ·
-
enhancement good first issue severity: low
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
TMHSDigital/Static_Chess#72 ·
-
documentation good first issue severity: low
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
TMHSDigital/Static_Chess#49 ·
TMHSDigital/Static_Chess の issue をすべて見る
似ている issue
-
難易度 1/5 1時間未満 初心者へのやさしさ 88/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 66/100
メンテナーはふだん 1 日以内に返信
-
discovery:other factory-active task-identify-harness-labels-done task-identify-issue-type-done
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
メンテナーはふだん 1 日以内に返信
-
needs-triage typo
難易度 1/5 1時間未満 初心者へのやさしさ 88/100
llm-d-incubation/llm-d-scaling-manager#122 ·
メンテナーはふだん 1 日以内に返信
-
create-strapi-app doesn't await writing license.txt, so a failed write skips its error handlingオープンflag: invalid template issue: bug severity: low source: cli status: pending reproduction version: 5
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
strapi/strapi#28637 · コメント 1 件 ·
メンテナーはふだん 2 日以内に返信