Skip to content

Conversation

@chiefenne
Copy link
Contributor

I had a hard time to figure everything out, so I thought it could be posted here.
Probably I forgot one or the other step, but the majority should be covered.
Some reformatting might be needed.
Best regards,
Andreas

I had a hard time to figure everything out, so I thought it could be posted here.
Probably I forgot one or the other step, but the majority should be covered.
Some reformatting might be needed.
Best regards,
Andreas
@economon
Copy link
Member

@chiefenne : thank you for the contribution! This is exactly what we had hoped for when opening up the docs to the community for contributions alongside the code.

I know that it is tough to see the formatting of the changes as they will look on the web (unless you can build the Jekyll locally), so we can accept this quickly and then let you follow up with any formatting changes you prefer.

We still need to improve the process a bit. If you have ideas about this, just let us know.

Copy link
Member

@economon economon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

Let's merge this in, and please just submit any updates as another PR. Thanks for the contribution!

@economon economon merged commit d9dc1e8 into su2code:master Mar 20, 2019
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