Closed grdryn closed 2 years ago
Ping @grdryn :)
@emmanuelbernard I thought I replied here, but clearly not. Sorry about that!
I created #46 to import the ADR (rather than a link to a non-public doc) so that it could be included here. There are a couple of things I wanted to try out on that PR, but I was unable to build the site locally (created #47). I meant to get back around to that to get a container image working that could be used to build the site locally...I'll try to move that closer to the top of the stack again :slightly_smiling_face:
@grdryn we are considering merging, anything more you want to put in this one?
@emmanuelbernard no, if you're happy, I'm happy :slightly_smiling_face: You'll have to merge it though, as I don't have that ability here.
@grdryn I left one suggested change.
@emmanuelbernard assuming you're happy, I'll merge this once @grdryn makes the suggested change.
I'm always happy ;)
@emmanuelbernard assuming you're happy, I'll merge this once @grdryn makes the suggested change.
@tombentley both of these conditions have been met now :slightly_smiling_face:
Issue: https://issues.redhat.com/browse/MGDX-33