Closed stickies-v closed 1 year ago
±0 on this. Checked that the table rows are now reversed in 8b46405bfb0753de5f556bf16c05eef479f51d87
I'd say rows are currently sorted in descending urgency.
It's gonna take many more paragraphs to convince me to ACK.
On Mon, Nov 14, 2022 at 5:47 PM stickies-v @.***> wrote:
It's a bit of a nit, but I find it a bit counterintuitive to put the oldest release at the top. Releases become less important the older they get, so this makes more sense to me.
You can view, comment on, or merge this pull request online at:
https://github.com/bitcoin-core/bitcoincore.org/pull/918 Commit Summary
- 8b46405 https://github.com/bitcoin-core/bitcoincore.org/pull/918/commits/8b46405bfb0753de5f556bf16c05eef479f51d87 Invert order of maintenance-table.md
File Changes
(1 file https://github.com/bitcoin-core/bitcoincore.org/pull/918/files)
- M _includes/posts/maintenance-table.md https://github.com/bitcoin-core/bitcoincore.org/pull/918/files#diff-39cef08af7a6ee4e98a23b05fb3011e308a6e2b6b630f31a2f05105d069d448b (10)
Patch Links:
- https://github.com/bitcoin-core/bitcoincore.org/pull/918.patch
- https://github.com/bitcoin-core/bitcoincore.org/pull/918.diff
— Reply to this email directly, view it on GitHub https://github.com/bitcoin-core/bitcoincore.org/pull/918, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMRS4WZQBG73N4RJDYV4YS3WIJUK3ANCNFSM6AAAAAASAAWWAA . You are receiving this because you are subscribed to this thread.Message ID: @.***>
~0
Looks like it's personal preference then. Apologies for wasting review time.
It's a bit of a nit, but I find it a bit counterintuitive to put the oldest release at the top. Releases become less important the older they get, so this makes more sense to me.