HomePhabricator

Resolve variant includes with the correct ordering

Description

Resolve variant includes with the correct ordering

Summary:
Refactored variant include resolution to correctly order the hierarchy
of includes while still protecting against infinite recursion. The
function was renamed, refactored for clarity, and exported so as to be
better tested and documented with examples.

Test Plan:
Run go test ./....

Check the rendered config.ResolveIncludes godoc for sanity.

Reviewers: thcipriani, mmodell, hashar, Jrbranaa, zeljkofilipin, demon, Release-Engineering-Team

Reviewed By: thcipriani, Release-Engineering-Team

Tags: Release-Engineering-Team

Differential Revision: https://phabricator.wikimedia.org/D959

Details

Provenance
dduvallAuthored on Jan 31 2018, 11:32 PM
Reviewer
Release-Engineering-Team
Differential Revision
D959: Resolve variant includes with the correct ordering
Parents
rGBLBR00820cbd6bbc: Use dep for dependency management and commit vendor
Branches
Unknown
Tags
Unknown
ChangeId
None