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

Make visitLineBreak return new line instead of space #633

Merged
merged 2 commits into from
Jun 27, 2023

Conversation

Kyle-Ye
Copy link
Contributor

@Kyle-Ye Kyle-Ye commented Jun 18, 2023

Bug/issue #, if applicable:

With the corresponding swift-docc-render PR, Close #477.

Summary

Make visitLineBreak return new line instead of space to align with CommonMark spec

Dependencies

  • swift-docc-render PR

Issue is tracked at swiftlang/swift-docc-render#697

Testing

Update RenderContentCompilerTests.swift

Checklist

Make sure you check off the following items. If they cannot be completed, provide a reason.

  • Added tests
  • Ran the ./bin/test script and it succeeded
  • Updated documentation if necessary

@Kyle-Ye
Copy link
Contributor Author

Kyle-Ye commented Jun 18, 2023

@swift-ci please test

@Kyle-Ye Kyle-Ye force-pushed the bugfix/kyle/hardbreak branch from b4ff394 to 59d1f6f Compare June 23, 2023 07:48
@Kyle-Ye Kyle-Ye force-pushed the bugfix/kyle/hardbreak branch from 59d1f6f to 16b212e Compare June 26, 2023 17:05
@Kyle-Ye
Copy link
Contributor Author

Kyle-Ye commented Jun 26, 2023

@swift-ci please test

@Kyle-Ye Kyle-Ye merged commit b549f1d into main Jun 27, 2023
@Kyle-Ye Kyle-Ye deleted the bugfix/kyle/hardbreak branch June 27, 2023 05:10
Kyle-Ye added a commit to Kyle-Ye/swift-docc that referenced this pull request Jul 11, 2023
* Make visitLineBreak return new line instead of space

* Update test case for more human-readable
d-ronnqvist pushed a commit that referenced this pull request Jul 12, 2023
* Make visitLineBreak return new line instead of space

* Update test case for more human-readable
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.

Hard breaks aren't reflected in output
2 participants