Add docstrings with version information to all evolutions.

Review Request #12290 — Created May 18, 2022 and submitted — Latest diff uploaded

Information

Review Board
release-5.0.x

Reviewers

Review Board has had a great many database schema changes over its
lifetime, and the more we add, the harder it is to determine when a
particular change was made.

This poses a problem when trying to sort out failed evolution upgrades,
as sometimes an upgrade is attempted and rolled back with evolution
information recorded that breaks future attempts at a new upgrade. When
this happens, it takes a lot of effort to figure out what happened.

This change updates all evolutions in the codebase to have a docstring
that specifies the version in which the evolution was added. This rounds
up to the first public release (so no alphas/betas).

All upgrades are also tracked in Notion, for easy reference:

https://reviewboard.notion.site/Database-Schema-Changes-38702981914e48a3b32f2a3dc5243a5c

While here, I found a stray evolution that was never used (its contents
were incorporated into another evolution). That's been deleted.

Went through and verified the versions of each of these.

Diff Revision 1 (Latest)

Commits

First Last Summary ID Author
Add docstrings with version information to all evolutions.
Review Board has had a great many database schema changes over its lifetime, and the more we add, the harder it is to determine when a particular change was made. This poses a problem when trying to sort out failed evolution upgrades, as sometimes an upgrade is attempted and rolled back with evolution information recorded that breaks future attempts at a new upgrade. When this happens, it takes a lot of effort to figure out what happened. This change updates all evolutions in the codebase to have a docstring that specifies the version in which the evolution was added. This rounds up to the first public release (so no alphas/betas). All upgrades are also tracked in Notion, for easy reference: https://reviewboard.notion.site/Database-Schema-Changes-38702981914e48a3b32f2a3dc5243a5c
c7a12978959812b5af81f7b01993858fff356e45 Christian Hammond
This diff has been split across 5 pages: 1 2 3 4 5 >
reviewboard/accounts/evolutions/extra_data.py
reviewboard/accounts/evolutions/is_private.py
reviewboard/accounts/evolutions/localsiteprofile_permissions.py
reviewboard/accounts/evolutions/open_an_issue.py
reviewboard/accounts/evolutions/profile_default_use_rich_text.py
reviewboard/accounts/evolutions/profile_settings.py
reviewboard/accounts/evolutions/profile_should_send_email.py
reviewboard/accounts/evolutions/profile_should_send_own_updates.py
reviewboard/accounts/evolutions/profile_show_closed.py
reviewboard/accounts/evolutions/reviewrequestvisit_visibility.py
reviewboard/accounts/evolutions/timezone.py
reviewboard/accounts/evolutions/timezone_length_30.py
reviewboard/accounts/evolutions/unique_together_baseline.py
reviewboard/attachments/evolutions/file_attachment_file_max_length_512.py
reviewboard/attachments/evolutions/file_attachment_orig_filename.py
reviewboard/attachments/evolutions/file_attachment_ownership.py
reviewboard/attachments/evolutions/file_attachment_repo_info.py
reviewboard/attachments/evolutions/file_attachment_repo_path_no_index.py
reviewboard/attachments/evolutions/file_attachment_repo_revision_max_length_64.py
reviewboard/attachments/evolutions/file_attachment_revision.py
This diff has been split across 5 pages: 1 2 3 4 5 >
Loading...