From 85f8d5aa99c168ee48d12ed936d68487373cd3da Mon Sep 17 00:00:00 2001 From: David Stansby Date: Mon, 6 May 2024 10:49:53 +0100 Subject: [PATCH 1/3] Improve affiliated package page structure --- affiliated.rst | 19 ++++++++----------- 1 file changed, 8 insertions(+), 11 deletions(-) diff --git a/affiliated.rst b/affiliated.rst index 2d7c882c..067f95c5 100644 --- a/affiliated.rst +++ b/affiliated.rst @@ -5,8 +5,8 @@ Affiliated Packages Affiliated packages are well-maintained, open source software packages that are useful to solar physicists and integrate well with the SunPy ecosystem. To aid discoverability, all affiliated packages are listed on the SunPy website, and the SunPy project tries to advertise them at national and international conferences and workshops. -Package list ------------- +Current packages +---------------- .. list-table:: :widths: 20, 80 @@ -414,14 +414,11 @@ Review Criteria and Summary | | provisional rating. | +---------------+-----------------------------------------------------+ -Open review process -^^^^^^^^^^^^^^^^^^^ +Becoming an affiliated package +------------------------------ -The review process for becoming a SunPy affiliated package is designed to be approachable, lightweight and open. -Reviews are conducted in GitHub issues through the https://github.com/sunpy/sunpy.org repository. - -Steps for Getting New Packages Reviewed -~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ +The review process for becoming an affiliated package is designed to be approachable, lightweight and open. +Reviews are conducted in GitHub issues through the https://github.com/sunpy/sunpy.org repository, using the following process: 1. If you're not sure whether to submit your package for the affiliated package review process, you can open an issue to informally discuss your package or contact the :ref:`role_affiliated-liaison` to discuss your package privately. 2. Open a new issue with the issue template. @@ -434,7 +431,7 @@ Steps for Getting New Packages Reviewed 9. The :ref:`role_affiliated-liaison` merges the pull request. Existing Packages Review Process -~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ +~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ Existing affiliated packages will be reviewed once per year by the :ref:`role_affiliated-liaison` to ensure the review is current. Developers may challenge a new review, which then requires the liaison to get an independent reviewer to perform the review. @@ -443,7 +440,7 @@ Existing provisional affiliated packages will be reviewed once per year by the : To pass they must not have a worse score and still be working towards meeting the rest of the review criteria. Acknowledgements -~~~~~~~~~~~~~~~~ +---------------- Sections of this page are heavily inspired by the `Astropy affiliated package review process `__. From 10312c0b412458f6b522e3e87a4da76d4037843c Mon Sep 17 00:00:00 2001 From: David Stansby Date: Mon, 6 May 2024 13:37:16 +0100 Subject: [PATCH 2/3] Pin astropy --- requirements.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/requirements.txt b/requirements.txt index bd234410..ee4c246a 100644 --- a/requirements.txt +++ b/requirements.txt @@ -12,7 +12,7 @@ sphinxext-opengraph sunpy-sphinx-theme>=2.0.0 # Eclipse notebook -astropy +astropy<6.1 exifread ipywidgets matplotlib From cd73fdc92c9914b2c2a3176e9cff321c1db9ca0d Mon Sep 17 00:00:00 2001 From: David Stansby Date: Mon, 6 May 2024 13:49:17 +0100 Subject: [PATCH 3/3] Fix title levels --- affiliated.rst | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/affiliated.rst b/affiliated.rst index 067f95c5..8ba4b456 100644 --- a/affiliated.rst +++ b/affiliated.rst @@ -431,7 +431,7 @@ Reviews are conducted in GitHub issues through the https://github.com/sunpy/sunp 9. The :ref:`role_affiliated-liaison` merges the pull request. Existing Packages Review Process -~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ +^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Existing affiliated packages will be reviewed once per year by the :ref:`role_affiliated-liaison` to ensure the review is current. Developers may challenge a new review, which then requires the liaison to get an independent reviewer to perform the review.