-
-
Notifications
You must be signed in to change notification settings - Fork 92
Fix GitHub Pages workflow #275
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from 3 commits
8a7439e
eb07258
dc35b6a
b95d8d7
2c47f0b
0246774
323d53e
7224385
1dae517
f0ca2ad
ebf1d88
95f81d5
19f9ff1
381f8f3
5457c00
517005a
991677f
8118b7c
576d3d8
608565c
49f6799
c61849d
f60fd95
0e35f56
03bca62
deb3890
28bd57d
c998b10
027211c
88281ce
625dc32
65899cb
2fe51d5
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -12,3 +12,6 @@ trim_trailing_whitespace = true | |
|
|
||
| [*.md] | ||
| trim_trailing_whitespace = false | ||
|
|
||
| [*.yml] | ||
| indent_size = 2 | ||
| Original file line number | Diff line number | Diff line change | ||||||||
|---|---|---|---|---|---|---|---|---|---|---|
| @@ -1,5 +1,3 @@ | ||||||||||
| defaults: | ||||||||||
| - scope: | ||||||||||
| path: "" | ||||||||||
| values: | ||||||||||
| render_with_liquid: false | ||||||||||
| - scope: | ||||||||||
| path: "" | ||||||||||
|
||||||||||
| path: "" | |
| path: "" | |
| values: | |
| render_with_liquid: false |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -7,7 +7,7 @@ | |
| msgid "" | ||
| msgstr "" | ||
| "Project-Id-Version: PACKAGE VERSION\n" | ||
| "POT-Creation-Date: 2025-10-19 11:32+0000\n" | ||
| "POT-Creation-Date: 2025-12-20 13:19+0000\n" | ||
| "PO-Revision-Date: YEAR-MO-DA HO:MI+ZONE\n" | ||
| "Last-Translator: FULL NAME <EMAIL@ADDRESS>\n" | ||
| "Language-Team: LANGUAGE <[email protected]>\n" | ||
|
|
@@ -47,9 +47,8 @@ msgstr "" | |
| #. type: Plain text | ||
| #: ../../guide/en/start/workflow.md | ||
| msgid "" | ||
| "The installed application has only the homepage, which displays when you " | ||
| "access the URL `http://localhost/`. It shares a common layout that you can " | ||
| "reuse on further pages." | ||
| "The installed application contains only one page, accessible at `http://" | ||
| "localhost/`. It shares a common layout that you can reuse on further pages." | ||
| msgstr "" | ||
|
|
||
| #. type: Plain text | ||
|
|
@@ -165,9 +164,24 @@ msgstr "" | |
| msgid "The following diagram shows how an application handles a request." | ||
| msgstr "" | ||
|
|
||
| #. type: Plain text | ||
| #. type: Fenced code block (mermaid) | ||
| #: ../../guide/en/start/workflow.md | ||
| msgid "" | ||
| #, no-wrap | ||
| msgid "" | ||
| "flowchart LR\n" | ||
| " user[User's client] --> index\n" | ||
| " index[index.php] --> DI[Initialize Dependency Container]\n" | ||
| " config[configs] -.-> DI\n" | ||
| " DI --> RequestFactory[RequestFactory]\n" | ||
| " RequestFactory -->|Request| app[Application]\n" | ||
| " app -->|Request| middleware[Middleware]\n" | ||
| " middleware -->|Request| router[Router]\n" | ||
| " router -->|Request| action[Action Handler]\n" | ||
| " action -->|Response| emitter[SapiEmitter]\n" | ||
| " router -->|Response| emitter\n" | ||
| " middleware -->|Response| emitter\n" | ||
| " app -->|Response| emitter\n" | ||
| " emitter --> user\n" | ||
| msgstr "" | ||
|
|
||
| #. type: Bullet: '1. ' | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The pull_request trigger is marked as "TEMPORARY !!!" but should be removed before merging to production. This trigger will cause the workflow to run on all pull requests, which is generally not desired for a GitHub Pages deployment workflow. Deployment to GitHub Pages should typically only happen on push to the main branch or via manual workflow_dispatch.