From 4fc640bfa2bfae8abb8272411b465c5a16f07213 Mon Sep 17 00:00:00 2001 From: Emily KL <4672118+emilykl@users.noreply.github.com> Date: Mon, 23 Feb 2026 15:30:31 -0500 Subject: [PATCH 1/5] Update pull_request_template.md --- .github/pull_request_template.md | 53 +++++++++++--------------------- 1 file changed, 18 insertions(+), 35 deletions(-) diff --git a/.github/pull_request_template.md b/.github/pull_request_template.md index 0ac4aea78e7..6f16e3e683b 100644 --- a/.github/pull_request_template.md +++ b/.github/pull_request_template.md @@ -1,38 +1,21 @@ + +## Link to issue + +Closes (#) + +## Description of change + +(1-2 sentences) + +## Demo + +(Include screenshot or screen recording) + +## Testing strategy + +(1-2 sentences explaining tests added or changed, or why testing changes are not needed) From c906710f7ff74ee24fe484dc0fb8547dd8840e23 Mon Sep 17 00:00:00 2001 From: Emily KL <4672118+emilykl@users.noreply.github.com> Date: Mon, 23 Feb 2026 15:39:24 -0500 Subject: [PATCH 2/5] Update pull_request_template.md --- .github/pull_request_template.md | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/.github/pull_request_template.md b/.github/pull_request_template.md index 6f16e3e683b..4865a4f4d70 100644 --- a/.github/pull_request_template.md +++ b/.github/pull_request_template.md @@ -5,17 +5,17 @@ Please complete each section below. --> ## Link to issue + -Closes (#) +Closes #(issue number) ## Description of change - -(1-2 sentences) + ## Demo -(Include screenshot or screen recording) + ## Testing strategy -(1-2 sentences explaining tests added or changed, or why testing changes are not needed) + From 54cddd57e751827e90c341ac84ab7a67904d8234 Mon Sep 17 00:00:00 2001 From: Emily KL <4672118+emilykl@users.noreply.github.com> Date: Wed, 25 Feb 2026 17:05:00 -0500 Subject: [PATCH 3/5] Add checklists removed from PR template into CONTRIBUTING.md --- CONTRIBUTING.md | 36 ++++++++++++++++++++++++++++++++++++ 1 file changed, 36 insertions(+) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index 9f5f4d636ad..b6d8fe7773d 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -171,6 +171,42 @@ and create your pull request. > Please do _not_ commit changes to `uv.lock` > unless you have added, removed, or changed dependencies in `pyproject.toml`. +## Opening a pull request + +When creating your pull request, please follow the guidelines below. + +### Code pull request + +- *Make sure you have reviewed the full [contributing notes (this file)](https://github.com/plotly/plotly.py/blob/main/CONTRIBUTING.md) and understand the structure of the package.* +- If your PR modifies code of `plotly.graph_objects`, the modifications should be made to the the code generator, *not* the generated files. +- You have added tests or modified existing tests, as needed. +- For a new feature, you have added documentation examples (please see the doc checklist as well). +- You have added a CHANGELOG entry if changing anything substantial. +- For a new feature or a change in behavior, you have updated the relevant docstrings in the code. + +### Documentation pull request + +- *Make sure you have reviewed the [`doc/README.md`](https://github.com/plotly/plotly.py/blob/main/doc/README.md) file.* +- This change runs in the current version of Plotly on PyPI and targets the `doc-prod` branch OR it targets the `main` branch. +- If this PR modifies the first example in a page or adds a new one, it is a `px` example if at all possible. +- Every new/modified example has a descriptive title and motivating sentence or paragraph. +- Every new/modified example is independently runnable. +- Every new/modified example is optimized for short line count and focuses on the Plotly/visualization-related aspects of the example rather than the computation required to produce the data being visualized. +- Meaningful/relatable datasets are used for all new examples instead of randomly-generated data where possible. +- The random seed is set if using randomly-generated data. +- New/modified remote datasets are loaded from https://plotly.github.io/datasets and added to https://github.com/plotly/datasets. +- Large computations are avoided in the new/modified examples in favour of loading remote datasets that represent the output of such computations. +- Imports are `plotly.graph_objects as go`, `plotly.express as px`, and/or `plotly.io as pio`. +- Data frames are always called `df`. +- `fig = ` is called high up in each new/modified example (either `px.` or `make_subplots` or `go.Figure`). +- Liberal use is made of `fig.add_*` and `fig.update_*` rather than `go.Figure(data=..., layout=...)`. +- Specific adders and updaters like `fig.add_shape` and `fig.update_xaxes` are used instead of big `fig.update_layout` calls. +- `fig.show()` is at the end of each example. +- `plotly.plot()` and `plotly.iplot()` are not used in any example. +- Named colors are used instead of hex codes wherever possible. +- Code blocks are marked with ````python`. + + ### Testing We use [pytest](https://docs.pytest.org/) for managing and running tests. From a482c16b57c87dbc81a56553c2f2cd6e95fa647c Mon Sep 17 00:00:00 2001 From: Emily KL <4672118+emilykl@users.noreply.github.com> Date: Wed, 25 Feb 2026 17:10:15 -0500 Subject: [PATCH 4/5] add "additional information" section, and link to guidelines --- .github/pull_request_template.md | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/.github/pull_request_template.md b/.github/pull_request_template.md index 4865a4f4d70..84220a2e800 100644 --- a/.github/pull_request_template.md +++ b/.github/pull_request_template.md @@ -19,3 +19,11 @@ Closes #(issue number) ## Testing strategy + +## Additional information (optional) + + + +## Guidelines + +- [ ] I have reviewed the [pull request guidelines](https://github.com/plotly/plotly.py/blob/update-pr-template/CONTRIBUTING.md#opening-a-pull-request) and confirmed that this PR follows them. From 5be243ecf14e7775274e80e0865faa70bda773bb Mon Sep 17 00:00:00 2001 From: Emily KL <4672118+emilykl@users.noreply.github.com> Date: Wed, 25 Feb 2026 17:11:55 -0500 Subject: [PATCH 5/5] fix link to CONTRIBUTING.md --- .github/pull_request_template.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/pull_request_template.md b/.github/pull_request_template.md index 84220a2e800..9b5e1bb5121 100644 --- a/.github/pull_request_template.md +++ b/.github/pull_request_template.md @@ -26,4 +26,4 @@ Closes #(issue number) ## Guidelines -- [ ] I have reviewed the [pull request guidelines](https://github.com/plotly/plotly.py/blob/update-pr-template/CONTRIBUTING.md#opening-a-pull-request) and confirmed that this PR follows them. +- [ ] I have reviewed the [pull request guidelines](https://github.com/plotly/plotly.py/blob/main/CONTRIBUTING.md#opening-a-pull-request) and confirmed that this PR follows them.