bartavelle / language-puppet

A library to work with Puppet manifests, test them and eventually replace everything ruby.
BSD 3-Clause "New" or "Revised" License
51 stars 8 forks source link

Edgemaps removed from WireCatalog in json #155

Closed PierreR closed 6 years ago

PierreR commented 8 years ago

I don't quite understand what the filter here is doing but it does wipe out all the edges from the json output for me. So in the json output, edges is always [] but if I remove the filter I get them back.

What does it mean to be a knownEdge ?

bartavelle commented 8 years ago

IIRC the edgemap should only contain edges that can't be inferred from the other parts of the catalog. This is experimental code though, and I only tested it a couple of times as a real puppet server (it seemed to work!).

PierreR commented 8 years ago

Ah, nice. Just push a commit that moves the utils function from Puppet.Interpreter.Type to a separate module Puppet.Interpreter.Utils Hope you don't mind ;-). The module was quite big with all these functions around.

bartavelle commented 8 years ago

Should be for the best.