Skip to content

Conversation

@llogiq
Copy link
Contributor

@llogiq llogiq commented Mar 10, 2020

This fixes #5280 by checking the language attribute on code blocks.


changelog: none

@llogiq
Copy link
Contributor Author

llogiq commented Mar 10, 2020

Also @Manishearth can we remove the Appveyor build from the repo?

@Manishearth
Copy link
Member

Done

@Manishearth
Copy link
Member

I didn't delete the project so we can keep the history, i just made it not trigger builds for anything anymore

@Manishearth
Copy link
Member

@bors r=flip1995,Manishearth

@bors
Copy link
Contributor

bors commented Mar 10, 2020

📌 Commit d5c8b78 has been approved by flip1995,Manishearth

@bors
Copy link
Contributor

bors commented Mar 10, 2020

⌛ Testing commit d5c8b78 with merge 23bd427...

@bors
Copy link
Contributor

bors commented Mar 10, 2020

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: flip1995,Manishearth
Pushing 23bd427 to master...

@bors bors merged commit 23bd427 into master Mar 10, 2020
@llogiq llogiq deleted the needless_doc_main_code branch March 10, 2020 22:19
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.

needless_doctest_main false positive with ```txt

5 participants