Skip to content

Commit

Permalink
Add documentation about workarounds
Browse files Browse the repository at this point in the history
Recommend opening an issue to check them.

See #2075.
  • Loading branch information
lmoureaux committed Dec 15, 2023
1 parent 8416eee commit c778696
Show file tree
Hide file tree
Showing 3 changed files with 17 additions and 0 deletions.
1 change: 1 addition & 0 deletions docs/Contributing/index.rst
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@ Freeciv21 project.
pull-request.rst
eval-pull-request.rst
release.rst
workarounds.rst
stable-branch.rst
style-guide.rst
:maxdepth: 1
5 changes: 5 additions & 0 deletions docs/Contributing/release.rst
Original file line number Diff line number Diff line change
Expand Up @@ -53,6 +53,11 @@ These are the general steps to prepare and finalize a release:
created PR's.
#. While inside the ``Release Update of AutoRevision.txt`` PR, the release manager will enable an automatic
rebase and merge.
#. The release manager will open an issue titled ``Review workarounds after <version> release`` with the following text:

We should review the workarounds in the source code and check that they are still needed. Some
workarounds are documented here: :doc:`Workarounds`.

#. When all the CI actions are complete, the release manager will make a post in the ``#news-channel`` on the
Longturn.net Discord server.
#. The release manager will download the Windows i686 and x86_64 installer packages and use their Microsoft
Expand Down
11 changes: 11 additions & 0 deletions docs/Contributing/workarounds.rst
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
.. SPDX-License-Identifier: GPL-3.0-or-later
.. SPDX-FileCopyrightText: Louis Moureaux <[email protected]>
Workarounds
***********

This page lists workarounds in the code. They should be checked from time to
time and removed once they are no longer needed.

#. The macOS CI installs a fixed Python version to work around a bug in Meson.
(``.github/workflows/build.yaml``)

0 comments on commit c778696

Please sign in to comment.