Declare string literal as frozen
ClosedPublic
Actions

Authored by dereckson on Sep 21 2018, 15:37.

Tags

None

Referenced Files

	F3912580: D1787.diff
	Thu, Dec 19, 06:16

	Unknown Object (File)
	Sun, Dec 15, 22:33

	Unknown Object (File)
	Fri, Dec 13, 22:39

	Unknown Object (File)
	Wed, Dec 11, 09:15

	Unknown Object (File)
	Sun, Dec 8, 17:49

	Unknown Object (File)
	Fri, Dec 6, 00:33

	Unknown Object (File)
	Wed, Dec 4, 18:36

	Unknown Object (File)
	Wed, Dec 4, 18:36

View All Files

Subscribers

None

Details

Reviewers

dereckson

Maniphest Tasks

T912: Follow the community Ruby Style Guide

Commits

rTOMMY5c4fffd347b1: Declare string literal as frozen

Summary

Ruby 3 will use frozen string litteral by default.

As Tommy strings are immutable, we can already switch to this behavior.

References:

Test Plan

Run server

Diff Detail

Repository

rTOMMY Tommy

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

dereckson requested review of this revision.Sep 21 2018, 15:37

dereckson created this revision.

Harbormaster completed remote builds in B2773: Diff 4531.Sep 21 2018, 15:37

dereckson accepted this revision.Sep 21 2018, 15:38

This revision is now accepted and ready to land.Sep 21 2018, 15:38

Closed by commit rTOMMY5c4fffd347b1: Declare string literal as frozen (authored by dereckson). · Explain WhySep 21 2018, 15:38

This revision was automatically updated to reflect the committed changes.

dereckson added a task: T912: Follow the community Ruby Style Guide.Sep 21 2018, 15:40

Revision Contents
Changeset List

Path

Size

Gemfile

2 lines

config.ru

2 lines

tommy.rb

2 lines

Diff 4532

View Options

Gemfile

View Options

config.ru

View Options

Declare string literal as frozenClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 4532

Gemfile

config.ru

tommy.rb

Declare string literal as frozen
ClosedPublic
Actions

Revision Contents
Changeset List