Skip to content

Fix a couple of rst warnings#615

Merged
sigmavirus24 merged 5 commits into
realpython:masterfrom
markvl:fix_warnings
Oct 21, 2015
Merged

Fix a couple of rst warnings#615
sigmavirus24 merged 5 commits into
realpython:masterfrom
markvl:fix_warnings

Conversation

@markvl
Copy link
Copy Markdown
Contributor

@markvl markvl commented Oct 20, 2015

No description provided.

Comment thread docs/scenarios/gui.rst Outdated
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why do we need that unicode character there? Why can't it just be a space?

@sigmavirus24
Copy link
Copy Markdown
Contributor

Please ping me when you've fixed those two items up.

@markvl
Copy link
Copy Markdown
Contributor Author

markvl commented Oct 21, 2015

I wanted to remove the warning without changing what was there too much. I assumed the unicode "non breaking space" was approved (required?) somewhere in the past for a reason.

@sigmavirus24 I have updated the first commit to remove the character completely.

@sigmavirus24
Copy link
Copy Markdown
Contributor

I don't know if they were approved or required. They were probably added without anyone noticing because GitHub doesn't display them in a way such that one might notice it. I'm guessing it was purely accidental and not intentional.

Thanks for fixing this up @markvl

sigmavirus24 added a commit that referenced this pull request Oct 21, 2015
Fix a couple of rst warnings
@sigmavirus24 sigmavirus24 merged commit 9f7d38e into realpython:master Oct 21, 2015
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