Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

APE 1: Number is gone once reserved (naming chaos fix 2) #97

Merged
merged 2 commits into from
Mar 4, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 9 additions & 4 deletions APE1.rst
Original file line number Diff line number Diff line change
Expand Up @@ -5,15 +5,18 @@ author: Perry Greenfield

date-created: 2013 October 11

date-last-revised: 2021 February 26
date-last-revised: 2024 February 5

date-accepted: 2013 November 8

type: Process

status: Accepted

revised-by: Lia Corrales, Thomas Robitaille, Erik Tollerud - 2021 February 26 - Added APE modification process
revised-by:

* Lia Corrales, Thomas Robitaille, Erik Tollerud - 2021 February 26 - Added APE modification process
* Pey Lian Lim - 2024 February 5 - Added APE numbering process

Abstract
--------
Expand Down Expand Up @@ -84,7 +87,8 @@ doing this.

Following a discussion on astropy-dev, the proposal should be submitted as a
pull request to astropy-APEs with the name APE<n>.rst where <n> is an
appropriately assigned number. The draft must use the APEtemplate.rst file.
appropriately assigned number (i.e., not already an accepted/proposed/rejected APE).
The draft must use the APEtemplate.rst file.
That a formal proposal has been submitted as a PR should be announced to the
astropy-dev list.

Expand Down Expand Up @@ -191,4 +195,5 @@ The coordinating committee thought it was a honking great idea.
Previous versions of this APE
-----------------------------

* 2013-11-08 [`DOI <http://doi.org/10.5281/zenodo.1043886>`_] [`GitHub <https://github.com/astropy/astropy-APEs/blob/42951733ac42c0ea178d8df30705274a43c93091/APE1.rst>`_]
* 2013-11-08 [`DOI <http://doi.org/10.5281/zenodo.1043886>`_] [`GitHub <https://github.com/astropy/astropy-APEs/blob/42951733ac42c0ea178d8df30705274a43c93091/APE1.rst>`_]
* 2021-03-09 [`GitHub <https://github.com/astropy/astropy-APEs/commit/f2afc2ec72575d3c592b3a5c6257117b07b5b755>`_]
6 changes: 3 additions & 3 deletions README.rst
Original file line number Diff line number Diff line change
Expand Up @@ -148,7 +148,7 @@ Finalizing APEs
^^^^^^^^^^^^^^^

The final decision on accepting or rejecting APEs lies with the Astropy
Coordination Committee. Once the community discussion on the APE has wound
Coordination Committee (CoCo). Once the community discussion on the APE has wound
down, the committee discusses the APE and makes a final decision on acceptance
or rejection. One of the committee members should then:

Expand All @@ -158,7 +158,7 @@ or rejection. One of the committee members should then:
#. Update the "date-last-revised" to the day of merging and "status" to
"Accepted" or "Rejected".
#. If necessary, rename the APE file to be ``APE##.rst``, where ## is the next
free number on the list of APEs.
free number on the list of APEs. Only do this if there is a conflict.
#. Leave a brief comment in the PR indicating the result.
#. Merge the PR with the above changes.
#. If the APE was accepted then continue with the remaining steps, otherwise
Expand Down Expand Up @@ -198,7 +198,7 @@ or rejection. One of the committee members should then:
Updating APEs
^^^^^^^^^^^^^

In the cases where an updated APE requires updating (e.g. references to a new
In the cases where an updated APE requires updating (e.g., references to a new
APE that supercedes it, clarifying information that emerges after the APE is
accepted, etc.), changes can be made directly via PR, but the
"date-last-revised" should be updated in the APE. Additionally, the Zenodo entry
Expand Down