Skip to content

Convert Library Evolution docs from ReST to Markdown. #369

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

Merged

Conversation

paulyoung
Copy link

This is the same as #363 but against the correct branch.

cc @jrose-apple

@jrose-apple
Copy link
Contributor

I'm a little worried about losing all the tables of contents. Some of these docs are quite big and it's unfortunate not to have them. On the other hand, having to specify them manually and keep them up to date would also be a loss.

Maybe we can add the Kramdown placeholder, so that someone actually rendering the docs using Kramdown will get one. (Why Kramdown? Because the Swift website is using Jekyll, and we could conceiveably put the docs there.)

} else {
print("BOO!!")
}
}

Choose a reason for hiding this comment

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

What about:

```swift
// Code here
```

In order to make it highlighted?

@paulyoung
Copy link
Author

Maybe we can add the Kramdown placeholder

@jrose-apple presumably that would be just after the title?

I'm also thinking that it wouldn't have its own heading since that would look strange on GitHub, but please let me know what you think.

@paulyoung
Copy link
Author

I think I've addressed all of the feedback so far. I'll check back later for anything further.

types. These are similar to the internal `effects` attribute we have for
functions, except that they can be enforced by the compiler.

- `trivial`: Promises that the type is trivial. Note that this is not a
Copy link
Contributor

Choose a reason for hiding this comment

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

Sorry, I just meant the attribute name didn't need to be linked. The word "trivial" in the first sentence should still be linked to #trivial.

Copy link
Author

Choose a reason for hiding this comment

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

Updated in b849183d854540a046e8a0ab0b56fee0397ff3d9. Please let me know if that's still not right.

@jrose-apple
Copy link
Contributor

Thanks, this all looks good! Can you squash it down to a single commit for the merge?

@paulyoung paulyoung force-pushed the library-evolution-md branch from b4ced9c to 5702493 Compare December 11, 2015 17:47
@paulyoung
Copy link
Author

Done!

@jrose-apple
Copy link
Contributor

All right, that's two down. Thanks, Paul!

jrose-apple added a commit that referenced this pull request Dec 11, 2015
Convert Library Evolution docs from ReST to Markdown.
@jrose-apple jrose-apple merged commit cf7558f into swiftlang:rst-to-markdown Dec 11, 2015
@paulyoung paulyoung deleted the library-evolution-md branch December 12, 2015 12:00
khizkhiz pushed a commit to khizkhiz/swift that referenced this pull request Mar 11, 2016
Convert Library Evolution docs from ReST to Markdown.
slavapestov pushed a commit to slavapestov/swift that referenced this pull request Nov 27, 2018
Report crash messages on Windows to the CRT
freak4pc pushed a commit to freak4pc/swift that referenced this pull request Sep 28, 2022
Remove unnecessary xfail for Kickstarter
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.

3 participants