Skip to content

Conversation

ValentijnvdBeek
Copy link
Contributor

Fixing the first item in my todo list in #338 which is to use code-blocks instead of literals to make it more readable. There are also a few more minor fixes which mainly revolve around making it easier to add it to the sphinx documentation.

Valentijn added 2 commits July 14, 2016 03:58
Use code-blocks instead of literals, change headings for portability and use a relative link to AUTHORS instead of linking to a specific copy.
@leandrotoledo leandrotoledo added the ⚙️ documentation affected functionality: documentation label Jul 14, 2016
@leandrotoledo leandrotoledo self-assigned this Jul 14, 2016
@leandrotoledo leandrotoledo added the 📋 pending-review work status: pending-review label Jul 14, 2016
@leandrotoledo
Copy link
Member

Hi @Faalentijn, thanks for this PR!
Review is in progress...

@@ -189,4 +189,4 @@ xml:
pseudoxml:
$(SPHINXBUILD) -b pseudoxml $(ALLSPHINXOPTS) $(BUILDDIR)/pseudoxml
@echo
@echo "Build finished. The pseudo-XML files are in $(BUILDDIR)/pseudoxml."
@echo "Build finished. The pseudo-XML files are in $(BUILDDIR)/pseudoxml."
Copy link
Member

Choose a reason for hiding this comment

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

Any specific reasons for EOF without a newline?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I was asked to do it by @tsnoam in the previous pull request.

Copy link
Member

Choose a reason for hiding this comment

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

LGTM

@leandrotoledo
Copy link
Member

One minor comment, everything else looks good.

@leandrotoledo
Copy link
Member

Looks nicer, thanks @Faalentijn. Merging.

@leandrotoledo leandrotoledo merged commit f5f95ef into python-telegram-bot:master Jul 14, 2016
@rahiel rahiel removed the 📋 pending-review work status: pending-review label Jul 14, 2016
@github-actions github-actions bot locked and limited conversation to collaborators Aug 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
⚙️ documentation affected functionality: documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants