|
2 | 2 |
|
3 | 3 | ## Summary |
4 | 4 |
|
5 | | -This release adds linting of code examples in *docstrings*, a workflow to check if PRs have updated the release notes and an [editorconfig](https://editorconfig.org/) file, as well as a bunch of bug fixes. |
| 5 | +<!-- Here goes a general summary of what this release is about --> |
6 | 6 |
|
7 | 7 | ## Upgrading |
8 | 8 |
|
9 | | -- nox: Now the default configuration for API repositories will not automatically add `pytests` as an `extra_path` |
10 | | - |
11 | | - The `pytests` directory is not a standard directory that will be auto-discovered by `pytest`, so it should always be included in the `pyproject.toml` file, in the `tool.pytest.ini_options.testpaths` array. Please check your API project is properly configured. |
| 9 | +<!-- Here goes notes on how to upgrade from previous versions, including deprecations and what they should be replaced with --> |
12 | 10 |
|
13 | 11 | ### Cookiecutter template |
14 | 12 |
|
15 | | -- To make the new workflow to check if release notes were updated you should add the check to the branch protection rules of your repository to require this check to pass. You should also add a new label *"cmd:skip-release-notes"* to be able to override the check. You can use the following script to do it: |
16 | | - |
17 | | - ```sh |
18 | | - repo=... # org/repo |
19 | | - token=... # GitHub token with the correct permissions |
20 | | - name="cmd:skip-release-notes" |
21 | | - desc="It is not necessary to update release notes for this PR" |
22 | | - color="930F79" |
23 | | - |
24 | | - # Using cURL |
25 | | - curl -L \ |
26 | | - -X POST \ |
27 | | - -H "Accept: application/vnd.github+json" \ |
28 | | - -H "Authorization: Bearer $token" \ |
29 | | - -H "X-GitHub-Api-Version: 2022-11-28" \ |
30 | | - -d '{"name":"'"$name"'","description":"'"$desc"'","color":"'"$color"'"}' \ |
31 | | - "https://api.github.com/repos/$repo/labels" |
32 | | - |
33 | | - # Using the gh tool (no need for a token if you already have it configured) |
34 | | - gh api -X POST \ |
35 | | - -f name="$name" -f description="$desc" -f color="$color" \ |
36 | | - "repos/$repo/labels" |
37 | | - ``` |
| 13 | +<!-- Here upgrade steps for cookiecutter specifically --> |
38 | 14 |
|
39 | 15 | ## New Features |
40 | 16 |
|
41 | | -- Add support for linting code examples found in *docstrings*. |
42 | | - |
43 | | - A new module `frequenz.repo.config.pytest.examples` is added with an utility function to be able to easily collect and lint code examples in *docstrings*. |
44 | | - |
45 | | - There is also a new optional dependency `extra-lint-examples` to easily pull the dependencies needed to do this linting. Please have a look at the documentation in the `frequenz.repo.config` package for more details. |
| 17 | +<!-- Here goes the main new features and examples or instructions on how to use them --> |
46 | 18 |
|
47 | 19 | ### Cookiecutter template |
48 | 20 |
|
49 | | -- Add a new GitHub workflow to check that release notes were updated. |
50 | | - |
51 | | - This workflow will check PRs to see if a change was done in the `src/` directory, and if so, it will fail if the `RELEASE_NOTES.md` wasn't also updated. |
52 | | - |
53 | | - Users can override this by assigning the label `cmd:skip-release-notes` to the PR for changes that don't really need a release notes update. |
54 | | - |
55 | | -- Add `MANIFEST.in` file. |
56 | | - |
57 | | - This makes sure that we don't ship useless files when building the distribution package and that we include all the relevant files too, like generated *.pyi files for API repositories. |
58 | | - |
59 | | -- Add an `.editorconfig` file to ensure a common basic editor configuration for different file types. |
60 | | - |
61 | | -- Add a `pytest` hook to collect and lint code examples found in *docstrings* using `pylint`. |
62 | | - |
63 | | - Examples found in code *docstrings* in the `src/` directory will now be collected and checked using `pylint`. This is done via the file `src/conftest.py`, which hooks into `pytest`, so to only check the examples you can run `pylint src`. |
64 | | - |
65 | | - !!! info |
66 | | - |
67 | | - There is a bug in the library used to extract the examples that prevents from collecting examples from `__init__.py` files. See https://github.com/frequenz-floss/frequenz-repo-config-python/issues/113 for more details. |
| 21 | +<!-- Here new features for cookiecutter specifically --> |
68 | 22 |
|
69 | 23 | ## Bug Fixes |
70 | 24 |
|
71 | | -- The distribution package doesn't include tests and other useless files anymore. |
72 | | - |
73 | | -- nox |
74 | | - |
75 | | - * When discovering path *extra paths*, now paths will not be added if they are also *source paths*, as we don't want any duplicates. |
76 | | - |
77 | | - * Fix copying of `Config` and `CommandOptions` objects. |
| 25 | +<!-- Here goes notable bug fixes that are worth a special mention or explanation --> |
78 | 26 |
|
79 | 27 | ### Cookiecutter template |
80 | 28 |
|
81 | | -- Now the CI workflow will checkout the submodules. |
82 | | - |
83 | | -- Fix adding of an empty keyword. |
84 | | - |
85 | | -- Don't distribute development files in the source distribution. |
| 29 | +<!-- Here bug fixes for cookiecutter specifically --> |
0 commit comments