Revert "Use hash symbols instead of strings for keys"
ClosedPublic
Actions

Authored by dereckson on Sep 21 2018, 21:48.

Tags

None

Referenced Files

	Unknown Object (File)
	Mon, Dec 15, 14:11

	Unknown Object (File)
	Mon, Dec 15, 05:38

	Unknown Object (File)
	Wed, Dec 10, 03:12

	Unknown Object (File)
	Mon, Dec 8, 10:39

	Unknown Object (File)
	Sun, Nov 30, 17:25

	Unknown Object (File)
	Sat, Nov 29, 17:03

	Unknown Object (File)
	Wed, Nov 26, 19:44

	Unknown Object (File)
	Nov 14 2025, 08:47

View All Files

Subscribers

None

Details

Reviewers

dereckson

Maniphest Tasks

T1449: Tommy doesn't show URL anymore

Commits

rTOMMY6da348fade1a: Revert "Use hash symbols instead of strings for keys"

Summary

Fix a regression where project properties aren't accessible anymore,
and so build URLs aren't included on the dashboard.

This reverts commit 517786ca4886.

Ref T1449

Test Plan

$ bundle exec rackup -o 0.0.0.0 -p 4567
$ curl localhost:4567 | grep ci

Diff Detail

Repository

rTOMMY Tommy

Lint

Lint Passed

Unit

No Test Coverage

Branch

fix-regression-url (branched from master)

Build Status

Buildable 2792
Build 3040: arc lint + arc unit

Event Timeline

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

dereckson created this revision.

Harbormaster completed remote builds in B2792: Diff 4564.Sep 21 2018, 21:48

dereckson edited the summary of this revision. (Show Details)Sep 21 2018, 21:49

dereckson accepted this revision.Sep 21 2018, 21:53

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

Closed by commit rTOMMY6da348fade1a: Revert "Use hash symbols instead of strings for keys" (authored by dereckson). · Explain WhySep 21 2018, 21:54

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

tommy.rb

6 lines

Diff 4564

View Options

Revert "Use hash symbols instead of strings for keys"ClosedPublicActions