Skip to content

Commit 9d0761e

Browse files
[TASK] Rework smaller parts to add specific notes (#339)
* [TASK] Rework smaller parts to add specific notes (This may use some :ref: links which are committed seperately) * Apply suggestions from code review --------- Co-authored-by: Stefan Frömken <[email protected]>
1 parent 2ba0550 commit 9d0761e

File tree

9 files changed

+55
-13
lines changed

9 files changed

+55
-13
lines changed

Documentation/AddingDocumentation/Index.rst

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -198,9 +198,8 @@ in :ref:`h2document:render-documentation-with-docker`.
198198

199199
.. important::
200200

201-
If you switch branches, you should rebuild everything. You can either remove the folder
202-
:file:`Documentation-GENERATED-temp/Cache` and all its contents or run `dockrun_t3rd
203-
makehtml-no-cache` which has the same effect.
201+
If you switch branches, you should rebuild everything. You can simply remove the folder
202+
:file:`Documentation-GENERATED-temp/Cache` and all its contents.
204203

205204
.. _documentation-main:
206205

Documentation/Appendix/HowToDeprecateThings.rst

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,12 @@ or change functionality in the TYPO3 Core, it has to be deprecated first.
1818
Upon deprecating or removing functionality the change must be added
1919
to the :ref:`Extension scanner <extension-scanner>` whenever possible.
2020

21+
.. note::
22+
It's not always possible to "deprecate" everything. In some cases,
23+
hard breaking changes must be introduced. These need to be addressed
24+
and discussed with much care beyond the scope of this chapter, and can
25+
only occur in new major versions of TYPO3 in the `X.0.x` focus release.
26+
2127
Here is how:
2228

2329

Documentation/Appendix/OSX/InstallGrunt.rst

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,11 @@
1010
GRUNT install on OSX
1111
====================
1212

13+
.. hint::
14+
15+
You may not need this at all, if you are using :ref:`runTests.sh <runtests_sh>`
16+
with dockerized environment for building all assets.
17+
1318
First off, make sure you installed **npm** like described :ref:`here<osx-npm>`.
1419

1520
In your Terminal app navigate to the folder that you cloned the TYPO3 source into. Switch into the ``Build`` folder and run

Documentation/Appendix/OSX/InstallNpm.rst

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,11 @@
1010
Npm install on OSX
1111
===================
1212

13+
.. hint::
14+
15+
You may not need this at all, if you are using :ref:`runTests.sh <runtests_sh>`
16+
with dockerized environment for building all assets.
17+
1318
Head over to https://docs.npmjs.com/cli/ and follow the install instructions.
1419

1520
Once you have installed it, you can open up the Terminal app which is located under Applications/Utilities on your mac.

Documentation/Build/Index.rst

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -35,3 +35,6 @@ It is also possible to lint the files locally:
3535
3636
Build/Scripts/runTests.sh -s lintScss
3737
Build/Scripts/runTests.sh -s lintTypescript
38+
39+
Remember to commit the compiled files alongside any patches, they are part
40+
of the monorepo.

Documentation/CheatSheets/Git.rst

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -132,6 +132,8 @@ This assumes, you have correctly configured your remote as described in
132132
133133
.. note::
134134
Pushing to `refs/publish` is deprecated, we now push to `refs/for`.
135+
Check out :ref:`git-commit-with-message` on how to specify a distinct
136+
small message alongsite your patch set.
135137

136138
.. _git-work-in-progress:
137139

Documentation/Community/Index.rst

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -45,8 +45,10 @@ any TYPO3 event in some way or another.
4545
* For an up-to-date list of events, see `Events on typo3.org
4646
<https://typo3.org/community/events/>`__
4747

48-
Look for events like "Review Friday", "Code Sprints" and "Developer Days".
49-
Next, we will describe some events that are relevant for core
48+
Look for events like "Review Friday", "Code Sprints" and "Developer Days"
49+
or one of the many "BarCamps".
50+
51+
Next, we will describe some events that are relevant for Core
5052
contribution. They are not specifically targeted at providing support,
5153
but the event may be open to new Core contributors and may give
5254
you the opportunity to ask questions and get help. In any case,

Documentation/ReportingAnIssue/Index.rst

Lines changed: 23 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,11 @@ Introduction to Forge
2424

2525
The issue tracker Forge is currently based on Redmine. It is used to
2626
report and handle open issues (including bugs and feature requests).
27+
28+
Note that Forge is not only used for the Community to report bugs
29+
and features, but also the Core development team creates issues for
30+
each and every change to TYPO3.
31+
2732
First, get yourself an account, see
2833
:ref:`TYPO3-Guide-ContributionWorkflow-Account`.
2934

@@ -49,6 +54,13 @@ something similar has already been reported.
4954
easy to find existing issues. Filters are located on the the left hand
5055
side of the navigation menu. You can use this feature to help refine searches.
5156

57+
Of course you are also able to use the search functionality of
58+
`Forge <https://forge.typo3.org>`__ itself, which offers some more specific
59+
filter options.
60+
61+
You help the people maintaining the issue tracker a lot by first making sure
62+
your problem is not covered already. The less duplicate issues our team needs
63+
to triage, the more time we have for actually addressing bugs and features.
5264

5365
.. index::
5466
single: Issue Reporting Workflow; Identifying
@@ -66,7 +78,8 @@ Remove side effects
6678
Work on a TYPO3 instance which is as clean as possible so you can
6779
rule out extensions messing with the TYPO3 Core. If you need to set
6880
up an extension to illustrate the problem, make sure it is as free
69-
of side effects as possible.
81+
of side effects as possible. Ideally, try to reproduce your
82+
problem or feature by using existing Core extensions (like :ref:`use-styleguide`).
7083

7184
Narrow down the problem
7285
Try different browsers, this will help the team (and you) a lot to
@@ -134,7 +147,8 @@ be using the most are **Feature** and **Bug**.
134147

135148
The others are mostly for internal organization (like **Stories** and **Epics**)
136149
and things which aren't really a feature or a bug... they are just **Tasks**
137-
somebody needs to take care of.
150+
somebody needs to take care of, because the Core development team also uses
151+
this system for their daily work.
138152

139153

140154

@@ -153,7 +167,8 @@ a report for yourself :)
153167
.. tip::
154168

155169
Remember, the subject is also helpful for people searching for already
156-
existing issues. Be as clear and precise as possible!
170+
existing issues. Be as clear and precise as possible! Also try to not mis-spell
171+
vital keywords (for example use "extbase" and not "ext-bse").
157172

158173
Bad example:
159174
Bug
@@ -298,7 +313,7 @@ It is important to provide **the specific information that is necessary** to be
298313
able to **understand**, **reproduce** and **fix** the bug.
299314

300315
Use your common sense and your experience to guide you: What would you need if you
301-
wanted to find the bug in the code and fix it? What do you need if you want to
316+
wanted to find the bug in the code and fix it yourself? What do you need if you want to
302317
reproduce it to debug it or test if the patch solves the problem?
303318

304319
A good bug report should contain all or any of these elements:
@@ -309,10 +324,10 @@ A good bug report should contain all or any of these elements:
309324

310325
* A brief description of your environment. Depending on the nature of
311326
the bug it *might* include your operating system
312-
(Windows, Linux, macOS, ...), the full version of TYPO3, the webserver
313-
used, the database used (mySql, SQLServer, ...) and its version.
314-
In any case, the full version of TYPO3 (e.g. 9.5.1) is very helpful. If
315-
your bug is reproducible on several versions (e.g. 10.1.0 and 9.5.1),
327+
(Windows, Linux, macOS, ...), the full version of TYPO3 and PHP, the webserver
328+
used, the database used (MySQL, MariaDB, ...) and its version.
329+
In any case, the full version of TYPO3 (e.g. 12.4.16) is very helpful. If
330+
your bug is reproducible on several versions (e.g. 13.2.1 and 12.4.16),
316331
that is helpful as well.
317332
* A description of the TYPO3 setup that you are using or that is necessary
318333
to trigger the bug. Your issue came out using TYPO3 with multiple

Documentation/Setup/Git/Index.rst

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -66,6 +66,9 @@ Clone the TYPO3 CMS core repository:
6666
6767
git clone https://github.com/typo3/typo3.git .
6868
69+
Of course you can also use your custom user-specific workspace like
70+
:file:`/home/kaspar/TYPO3-Contribution/` to store the files, as you
71+
do not necessarily need to run a webserver to later setup your installation.
6972

7073

7174
.. index::
@@ -196,6 +199,8 @@ when you do `git push`:
196199
197200
git config remote.origin.push +refs/heads/main:refs/for/main
198201
202+
Reminder: The TYPO3 source repository is hosted on GitLab. That Git repository is only mirrored to the TYPO3 GitHub
203+
repository. Gerrit is coupled to GitLab.
199204

200205
.. index::
201206
single: Code Contribution Workflow; Commit Message Template

0 commit comments

Comments
 (0)