Homec4science

Increase the storage size for commit summaries

Authored by epriestley <git@epriestley.com> on Aug 10 2016, 18:22.

Description

Increase the storage size for commit summaries

Summary:
Fixes T11453. Currently, commit message summaries are limited to 80 bytes. This may only be 20-40 characters for CJK languages or langauges with Cyrillic script.

Increase storage size to 255, then truncate to the shorter of 255 bytes or 80 glyphs. This preserves the same behavior for latin languages, but is less tight for Russian, etc.

Some minor additional changes:

  • Provide a way to ask "how much data fits in this column?" so we don't have to duplicate column lengths across summary checks or UI errors like "title too long".
  • Remove the text80 datatype, since no other columns use it and we have no use cases (or likely use cases) for it.

Test Plan:

  • Made a commit with a Cyrillic title, saw reasonable summarization in UI:

{F1757522}

  • Added and ran unit tests.
  • Grepped for removed SUMMARY_MAX_LENGTH constant.
  • Grepped for removed text80 data type.

Reviewers: avivey, chad

Reviewed By: avivey

Subscribers: avivey

Maniphest Tasks: T11453

Differential Revision: https://secure.phabricator.com/D16385

Details

Committed
epriestley <git@epriestley.com>Aug 10 2016, 20:12
Pushed
aubortJan 31 2017, 17:16
Parents
rPH38403b12beb6: Update Herald documentation for modern policies and beahvior
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPHe8083ad63a32: Increase the storage size for commit summaries (authored by epriestley <git@epriestley.com>).Aug 10 2016, 20:12