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

Issue Trees #45

Closed
StoneCypher opened this issue Dec 24, 2018 · 5 comments
Closed

Issue Trees #45

StoneCypher opened this issue Dec 24, 2018 · 5 comments

Comments

@StoneCypher
Copy link
Owner

image

https://en.wikipedia.org/wiki/Issue_tree#/media/File:An_issue_tree_showing_how_a_company_can_increase_profitability.png

@machinshin
Copy link
Collaborator

Not much to do here in india, so here's the fsl code (attached)

  1. where (and in what repo) do you want this example file uploaded?

  2. there are couple of inconsistencies between the png & the code
    i. the png shows some words in colored-bold, is that possible in fsl? if not, how hard would it be to add (maybe some kind of embedded markdown?)?
    ii. how/can do i put a box around a set of items?
    iii. should be possible to define the rendered png orientation?

  3. found 2 things i'd like to make writing fsl easier (at least in the live editor)
    i. in the fsl code it's hard to track levels ( i manually added L# in the comments)
    ii. I wish I could've used variables, cause atm all the strings make it hard to track. I believe that's in the roadmap, neh?

tree.txt

@machinshin machinshin added this to the Basic example machines milestone Jan 15, 2020
@StoneCypher
Copy link
Owner Author

This is awesome. Thank you

You deserve contributor credit for this. Open a PR putting it here: https://github.com/StoneCypher/jssm/tree/master/src/machines

@StoneCypher
Copy link
Owner Author

@machinshin - you didn't open the pr, you just merged to your fork

@StoneCypher StoneCypher reopened this Jan 26, 2020
@machinshin
Copy link
Collaborator

@StoneCypher : it merged the commit to my fork, then created this PR: StoneCypher/jssm#393
but github makes those two actions equivalent >:( . please merge that PR and let's close this one out.

@StoneCypher
Copy link
Owner Author

Oh heck, I was looking at the wrong thing :)

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

No branches or pull requests

2 participants