[coreboot-gerrit] [XL] Change in coreboot[master]: util/docker/coreboot.org-status: Rewrite parser

[Maxim Polyakov (Code Review)]

Attention is currently required from: Patrick Georgi, Martin L Roth, ron minnich.

Maxim Polyakov has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/59958 )

Change subject: util/docker/coreboot.org-status: Rewrite parser
......................................................................

Patch Set 17: Code-Review+2

(2 comments)

File util/docker/coreboot.org-status/board-status.html/boards.go:

https://review.coreboot.org/c/coreboot/+/59958/comment/7698f603_464f97d1 
PS17, Line 587: /*
              : 			venboard := vendor + "/" + board
              : 			if bi["Clone of"] != "" {
              : 				venboard = bi["Clone of"]
              : 			}
              : 		*/
Is it still needed? If so, maybe add some short comment about it?

File util/docker/coreboot.org-status/board-status.html/status-to-html.go:

https://review.coreboot.org/c/coreboot/+/59958/comment/ab3cd885_f7df3c49 
PS15, Line 71: dirs
> I used it more as a structuring mechanism, akin to unix pipes. […]

Ok, thanks for clarifying