Page MenuHomeDevCentral
Differential D1797

Ensure JENKINS_URL is present at app start
ClosedPublic
Actions

Authored by dereckson on Sep 21 2018, 18:49.
Tags
None
Referenced Files
F10172222: D1797.id4552.diff
Sun, Jun 22, 09:42
F10172210: D1797.id4551.diff
Sun, Jun 22, 09:41
F10171222: D1797.id4551.diff
Sun, Jun 22, 06:51
F10171149: D1797.diff
Sun, Jun 22, 06:46
Unknown Object (File)
Thu, Jun 19, 16:45
Unknown Object (File)
Fri, Jun 13, 18:30
Unknown Object (File)
Fri, Jun 13, 18:26
Unknown Object (File)
Thu, Jun 12, 00:13
View All Files
Subscribers
None

Details

Reviewers
dereckson
Commits
rTOMMY32f0d9106317: Ensure JENKINS_URL is present at app start
Summary

As configuration is mandatory, the previous solution to define a default value
can be improved by an explicit guidance about how to solve the issue.

Test Plan

Run server with and without JENKINS_URL

Diff Detail

Repository
rTOMMY Tommy
Lint
Lint Passed
Unit
No Test Coverage
Branch
env (branched from master)
Build Status
Buildable 2784
Build 3032: arc lint + arc unit

Event Timeline

dereckson requested review of this revision.Sep 21 2018, 18:49
dereckson created this revision.
Harbormaster completed remote builds in B2784: Diff 4551.Sep 21 2018, 18:49
dereckson accepted this revision.Sep 21 2018, 19:07
This revision is now accepted and ready to land.Sep 21 2018, 19:07
Closed by commit rTOMMY32f0d9106317: Ensure JENKINS_URL is present at app start (authored by dereckson). · Explain WhySep 21 2018, 19:07
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
7 lines
10 lines

Diff 4551

View Options

README.md

Loading...
View Options

tommy.rb

Loading...
Nasqueron DevCentral · If it had been much bigger the moon would have had a core of ice. · Powered by Phabricator