From a76ea22f68157f1d112070b88633ed1234cfc851 Mon Sep 17 00:00:00 2001 From: Charles Beauville Date: Wed, 6 Dec 2023 19:06:59 +0100 Subject: [PATCH] Add appendix --- ...tributor-tutorial-contribute-on-github.rst | 98 ++++++++++++++++--- 1 file changed, 85 insertions(+), 13 deletions(-) diff --git a/doc/source/contributor-tutorial-contribute-on-github.rst b/doc/source/contributor-tutorial-contribute-on-github.rst index 4422ce692e93..a94680ab526b 100644 --- a/doc/source/contributor-tutorial-contribute-on-github.rst +++ b/doc/source/contributor-tutorial-contribute-on-github.rst @@ -46,7 +46,7 @@ Setting up the repository $ git clone - This will create a `flower/` (or the name of your fork if you renamed it) folder in the current working directory. + This will create a ``flower/`` (or the name of your fork if you renamed it) folder in the current working directory. 4. **Add origin** You can then go into the repository folder: @@ -180,9 +180,9 @@ Creating and merging a pull request (PR) .. image:: _static/compare_and_pr.png - Otherwise you can always find this option in the `Branches` page. + Otherwise you can always find this option in the ``Branches`` page. - Once you click the `Compare & pull request` button, you should see something similar to this: + Once you click the ``Compare & pull request`` button, you should see something similar to this: .. image:: _static/creating_pr.png @@ -196,8 +196,8 @@ Creating and merging a pull request (PR) We have placed comments (that won't be rendered once the PR is opened) to guide you through the process. It is important to follow the instructions described in comments. For instance, in order to not break how our changelog system works, - you should read the information above the `Changelog entry` section carefully. Essentially, in this section, - you should put the description of your changes that will be added to the changelog alongside your PR title. + you should read the information above the ``Changelog entry`` section carefully. + You can also checkout some examples and details in the `appendix <#changelog-entry>`_ of this page. At the bottom you will find the button to open the PR. This will notify reviewers that a new PR has been opened and that they should look over it to merge or to request changes. @@ -276,8 +276,8 @@ Solution This is a tiny change, but it’ll allow us to test your end-to-end setup. After cloning and setting up the Flower repo, here’s what you should do: -- Find the source file in `doc/source` -- Make the change in the `.rst` file (beware, the dashes under the title should be the same length as the title itself) +- Find the source file in ``doc/source`` +- Make the change in the ``.rst`` file (beware, the dashes under the title should be the same length as the title itself) - Build the docs and check the result: ``_ Rename file @@ -288,18 +288,18 @@ If we just change the file, then we break all existing links to it - it is **ver Here’s how to change the file name: -- Change the file name to `save-progress.rst` -- Add a redirect rule to `doc/source/conf.py` +- Change the file name to ``save-progress.rst`` +- Add a redirect rule to ``doc/source/conf.py`` -This will cause a redirect from `saving-progress.html` to `save-progress.html`, old links will continue to work. +This will cause a redirect from ``saving-progress.html`` to ``save-progress.html``, old links will continue to work. Apply changes in the index file ::::::::::::::::::::::::::::::: -For the lateral navigation bar to work properly, it is very important to update the `index.rst` file as well. +For the lateral navigation bar to work properly, it is very important to update the ``index.rst`` file as well. This is where we define the whole arborescence of the navbar. -- Find and modify the file name in `index.rst` +- Find and modify the file name in ``index.rst`` Open PR ::::::: @@ -335,7 +335,7 @@ Here are a few positive examples which provide helpful information without repea * Update docs banner to mention Flower Summit 2023 * Remove unnecessary XGBoost dependency * Remove redundant attributes in strategies subclassing FedAvg -* Add CI job to deploy the staging system when the `main` branch changes +* Add CI job to deploy the staging system when the ``main`` branch changes * Add new amazing library which will be used to improve the simulation engine @@ -345,3 +345,75 @@ Next steps Once you have made your first PR, and want to contribute more, be sure to check out the following : - `Good first contributions `_, where you should particularly look into the :code:`baselines` contributions. + + +Appendix +-------- + +.. _changelog-entry: + +Changelog entry +::::::::::::::: + +When opening a new PR, inside its description, there should be a ``Changelog entry`` header. + +As per the comment above this section:: + + Inside the following 'Changelog entry' section, + you should put the description of your changes that will be added to the changelog alongside your PR title. + + If the section is completely empty (without any token), + the changelog will just contain the title of the PR for the changelog entry, without any description. + If the 'Changelog entry' section is removed entirely, + it will categorize the PR as "General improvement" and add it to the changelog accordingly. + If the section contains some text other than tokens, it will use it to add a description to the change. + If the section contains one of the following tokens it will ignore any other text and put the PR under the corresponding section of the changelog: + + is for classifying a PR as a general improvement. + is to not add the PR to the changelog + is to add a general baselines change to the PR + is to add a general examples change to the PR + is to add a general sdk change to the PR + is to add a general simulations change to the PR + + Note that only one token should be used. + +Its content must have a specific format. We will break down what each possibility does: + +- If the ``### Changelog entry`` section is removed, the following text will be added to the changelog:: + + - **General improvements** ([#PR_NUMBER](https://github.com/adap/flower/pull/PR_NUMBER)) + +- If the ``### Changelog entry`` section contains nothing but exists, the following text will be added to the changelog:: + + - **PR TITLE** ([#PR_NUMBER](https://github.com/adap/flower/pull/PR_NUMBER)) + +- If the ``### Changelog entry`` section contains a description (and no token), the following text will be added to the changelog:: + + - **PR TITLE** ([#PR_NUMBER](https://github.com/adap/flower/pull/PR_NUMBER)) + + DESCRIPTION FROM THE CHANGELOG ENTRY + +- If the ``### Changelog entry`` section contains ````, nothing will change in the changelog. + +- If the ``### Changelog entry`` section contains ````, the following text will be added to the changelog:: + + - **General improvements** ([#PR_NUMBER](https://github.com/adap/flower/pull/PR_NUMBER)) + +- If the ``### Changelog entry`` section contains ````, the following text will be added to the changelog:: + + - **General updates to Flower Baselines** ([#PR_NUMBER](https://github.com/adap/flower/pull/PR_NUMBER)) + +- If the ``### Changelog entry`` section contains ````, the following text will be added to the changelog:: + + - **General updates to Flower Examples** ([#PR_NUMBER](https://github.com/adap/flower/pull/PR_NUMBER)) + +- If the ``### Changelog entry`` section contains ````, the following text will be added to the changelog:: + + - **General updates to Flower SDKs** ([#PR_NUMBER](https://github.com/adap/flower/pull/PR_NUMBER)) + +- If the ``### Changelog entry`` section contains ````, the following text will be added to the changelog:: + + - **General updates to Flower Simulations** ([#PR_NUMBER](https://github.com/adap/flower/pull/PR_NUMBER)) + +Note that only one token must be provided, otherwise, only the first action (in the order listed above), will be performed. \ No newline at end of file