Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WIP: Diagrams from YAML #8

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

denysvitali
Copy link

As briefly discussed here, this PR represents a work in progress that I'm making on my own fork to handle the parsing of a YAML file, so that we can keep both paradigms: Infrastructure (Diagram) as Code, and Diagram fron YAML (kind of à la graphviz, but easier and tailored to infrastructure diagrams).

The idea is represented in the YAML file that this PR adds in its tests: unfortunately I still haven't figured out what's the best way to parse the type from the YAML file and convert it to a Golang type without creating a N elements switch that will need to be updated every time that we add a new icon 😅.

Feel free to improve the code as you wish: for the moment I've only provided the structure for this change and I'll hoepfully improve my PR in the following days

@denysvitali denysvitali changed the title WIP: Diagrams from YML WIP: Diagrams from YAML Sep 29, 2020
@tntobias
Copy link
Collaborator

tntobias commented Oct 1, 2020

Looking good so far! I will have some pretty major (breaking) changes coming in the next few days that will require this be reworked a touch. Good news is it should be much easier to get where you wanna go.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants