These are chat archives for assetgraph/assetgraph

6th
Jun 2017
Andreas Lind
@papandreou
Jun 06 20:54
@Munter Hmm, do you remember why this test expects the two JsonUrl relations to be there (without including unresolved) after populate has run? https://github.com/assetgraph/assetgraph/blob/b362ce0d8d4161fb20b92cca5283366026a0ae05/test/assets/ApplicationManifest.js#L32
The two images don't exist in testdata
Peter Müller
@Munter
Jun 06 20:57
Can't recall if that was on purpose. Probably not
Andreas Lind
@papandreou
Jun 06 21:02
Okay, thanks, I'll fix it. That seems like a bad side effect of populate.
Andreas Lind
@papandreou
Jun 06 21:09
I'm (slowly) working my way through assetgraph/assetgraph#730
Peter Müller
@Munter
Jun 06 21:13
Oooh, that is a nice one.
Andreas Lind
@papandreou
Jun 06 21:14
Yes, it could turn out to be nice, I think it could be an important bit in improving the API
Peter Müller
@Munter
Jun 06 21:20
Plus iron out a few of those weird edge cases we had with order of discovery
Andreas Lind
@papandreou
Jun 06 21:20
Currently fighting the async/sync monster