Skip to content

Commit 2e76301

Browse files
committed
minor #21288 Clarify renaming arguments isn't generally covered by the BC promise (janklan)
This PR was submitted for the 7.3 branch but it was merged into the 6.4 branch instead. Discussion ---------- Clarify renaming arguments isn't generally covered by the BC promise Based on a brief discussion in Slack, I believe it may be worth spelling out that argument names aren't covered by the BC promise. Commits ------- 1fe21d4 Clarify renaning arguments isn't generally covered by the BC promise
2 parents 3e74eb1 + 1fe21d4 commit 2e76301

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

contributing/code/bc.rst

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -285,6 +285,7 @@ Make final No
285285
Move to parent class Yes
286286
:ref:`Add argument without a default value <add-argument-public-method>` No
287287
:ref:`Add argument with a default value <add-argument-public-method>` No :ref:`[7] <note-7>` :ref:`[8] <note-8>`
288+
Rename argument Yes :ref:`[10] <note-10>`
288289
Remove argument No :ref:`[3] <note-3>`
289290
Add default value to an argument No :ref:`[7] <note-7>` :ref:`[8] <note-8>`
290291
Remove default value of an argument No
@@ -304,6 +305,7 @@ Make public No
304305
Move to parent class Yes
305306
:ref:`Add argument without a default value <add-argument-public-method>` No
306307
:ref:`Add argument with a default value <add-argument-public-method>` No :ref:`[7] <note-7>` :ref:`[8] <note-8>`
308+
Rename argument Yes :ref:`[10] <note-10>`
307309
Remove argument No :ref:`[3] <note-3>`
308310
Add default value to an argument No :ref:`[7] <note-7>` :ref:`[8] <note-8>`
309311
Remove default value of an argument No :ref:`[7] <note-7>`
@@ -320,6 +322,7 @@ Change name Yes
320322
Make public or protected Yes
321323
Add argument without a default value Yes
322324
Add argument with a default value Yes
325+
Rename argument Yes
323326
Remove argument Yes
324327
Add default value to an argument Yes
325328
Remove default value of an argument Yes

0 commit comments

Comments
 (0)