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/reviews/evolutions/group_mailing_list_charfield.py
reviewboard/reviews/evolutions/group_visible.py
reviewboard/reviews/evolutions/is_default_group.py
reviewboard/reviews/evolutions/last_review_timestamp.py
reviewboard/reviews/evolutions/localsite.py
reviewboard/reviews/evolutions/manytomanyfield_rm_null.py
reviewboard/reviews/evolutions/null_repository.py
reviewboard/reviews/evolutions/review_request_depends_on.py
reviewboard/reviews/evolutions/review_request_draft_commit_id.py
reviewboard/reviews/evolutions/review_request_file_attachment_histories.py
reviewboard/reviews/evolutions/review_request_issue_counts.py
reviewboard/reviews/evolutions/review_request_last_review_activity_timestamp.py
reviewboard/reviews/evolutions/review_request_screenshot_attachment_counters.py
reviewboard/reviews/evolutions/review_request_summary_index.py
reviewboard/reviews/evolutions/review_request_summary_index_manual.py
reviewboard/reviews/evolutions/reviewrequest_unique_together_baseline.py
reviewboard/reviews/evolutions/rich_text.py
reviewboard/reviews/evolutions/shipit_count.py
reviewboard/reviews/evolutions/split_rich_text.py
reviewboard/reviews/evolutions/status_update_timeout.py
This diff has been split across 5 pages: < 1 2 3 4 5 >
Loading...