Skip to content

Update or-gotcha.md#1046

Merged
SebastiaanZ merged 6 commits into
python-discord:masterfrom
ItsCinnabar:patch-1
Jul 16, 2020
Merged

Update or-gotcha.md#1046
SebastiaanZ merged 6 commits into
python-discord:masterfrom
ItsCinnabar:patch-1

Conversation

@ItsCinnabar
Copy link
Copy Markdown
Contributor

Adjust description and include link to docs

Adjust description and include link to docs
@ItsCinnabar ItsCinnabar requested a review from a team as a code owner July 14, 2020 13:48
@ItsCinnabar ItsCinnabar requested review from fiskenslakt and kwzrd and removed request for a team July 14, 2020 13:48
@ghost ghost added the needs 2 approvals label Jul 14, 2020
@ghost
Copy link
Copy Markdown

ghost commented Jul 14, 2020

Thank you for contributing to Python Discord!

Please check out the following documents:

@Numerlor
Copy link
Copy Markdown
Contributor

Discord mobile markdown is a bit messed up and in text links, the formatting gets ignored.

Would it make sense to only use "In Python" as the link portion?

SebastiaanZ
SebastiaanZ previously approved these changes Jul 14, 2020
Copy link
Copy Markdown
Contributor

@SebastiaanZ SebastiaanZ left a comment

Choose a reason for hiding this comment

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

Alright, looking good.

@ghost ghost removed the needs 1 approval label Jul 14, 2020
@SebastiaanZ SebastiaanZ dismissed their stale review July 14, 2020 15:08

formatting

@ghost ghost added the needs 1 approval label Jul 14, 2020
@SebastiaanZ
Copy link
Copy Markdown
Contributor

I can't replicate the formatting issue on android, which means I can't experiment with it. I'm wondering if this would give better results:

In Python, you should have _[complete instructions on both sides of the logical operator](https://docs.python.org/3/reference/expressions.html#boolean-operations)_.

Copy link
Copy Markdown
Contributor

@SebastiaanZ SebastiaanZ left a comment

Choose a reason for hiding this comment

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

There's a formatting issue on the android client. I'd normally not go out of my way to solve interclient issues, but there's an issue fix that should not influence anything from a semantic point of view.

Comment thread bot/resources/tags/or-gotcha.md Outdated
@ghost ghost added s: waiting for author Waiting for author to address a review or respond to a comment and removed needs 1 approval labels Jul 14, 2020
Co-authored-by: Sebastiaan Zeeff <33516116+SebastiaanZ@users.noreply.github.com>
@ghost ghost added needs 1 approval and removed s: waiting for author Waiting for author to address a review or respond to a comment labels Jul 14, 2020
Copy link
Copy Markdown
Contributor

@kwzrd kwzrd left a comment

Choose a reason for hiding this comment

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

Can confirm this now looks fine on Android. On iOS though, no cursive.

Which is fine, I think.

@ghost ghost removed the needs 1 approval label Jul 14, 2020
@kwzrd kwzrd added a: tags Related to bot tags p: 2 - normal Normal Priority labels Jul 14, 2020
Copy link
Copy Markdown
Member

@Den4200 Den4200 left a comment

Choose a reason for hiding this comment

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

It looks fine to me.

@Den4200 Den4200 requested a review from SebastiaanZ July 15, 2020 22:14
@SebastiaanZ SebastiaanZ merged commit 26317fc into python-discord:master Jul 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a: tags Related to bot tags p: 2 - normal Normal Priority

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants