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

Update docs for CI #636

Merged
merged 13 commits into from
Mar 10, 2021
Merged

Update docs for CI #636

merged 13 commits into from
Mar 10, 2021

Conversation

wolf99
Copy link
Contributor

@wolf99 wolf99 commented Mar 6, 2021

There was a question on Slack about the CI and specifically about the decision to stick with Ubuntu 18.04.
So I have taken the opportunity to update the docs on these points as lots has changed.

The docs will probably need changing a bit for v3 anyway, but I figure this information will be the same at least.

@wolf99 wolf99 requested a review from a team March 6, 2021 20:39
```
+-- <my-exercise>
<pre>
+-- &lt;my-exercise&gt;
Copy link
Member

Choose a reason for hiding this comment

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

Do you need to use &lt; below as well? The rich diff shows that it's messed up as-is.

docs/CONTRIBUTING.md Outdated Show resolved Hide resolved
wolf99 and others added 10 commits March 9, 2021 14:56
@wolf99
Copy link
Contributor Author

wolf99 commented Mar 9, 2021

Already had another branch with work done to lint all the docs with some changes that fixed the angle brackets issue.
So I've just merged and pushed that here. It's a bit more to review though!

Next step may be to add markdown lint CI...
(a la exercism/website-copy#1903)

docs/C_STYLE_GUIDE.md Outdated Show resolved Hide resolved
docs/C_STYLE_GUIDE.md Outdated Show resolved Hide resolved
@ryanplusplus ryanplusplus merged commit 5146d55 into main Mar 10, 2021
@ryanplusplus ryanplusplus deleted the ci-doc branch March 10, 2021 01:48
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