Skip to content

Added dracula theme style#1636

Merged
Anteru merged 4 commits intopygments:masterfrom
danvilar:master
Dec 24, 2020
Merged

Added dracula theme style#1636
Anteru merged 4 commits intopygments:masterfrom
danvilar:master

Conversation

@danvilar
Copy link
Copy Markdown
Contributor

Related to issue #1631

@danvilar
Copy link
Copy Markdown
Contributor Author

Corrected a column that was missing

@danvilar
Copy link
Copy Markdown
Contributor Author

I need some help with the failed check

@Leistungsabfall
Copy link
Copy Markdown
Contributor

Try force pushing your last commit

@danvilar
Copy link
Copy Markdown
Contributor Author

Try force pushing your last commit

I'm not sure how to do it. I did git push --force on my fork, but it just says "Everything up-to-date". I apologise in advance, I'm not used to doing it.

@Leistungsabfall
Copy link
Copy Markdown
Contributor

Run git commit --amend and then close nano (or whatever your default text editor is) without saving. Then execute git push --force.

@danvilar
Copy link
Copy Markdown
Contributor Author

Thank you @Leistungsabfall! I never used amends before. Nice to know for the future!

@Leistungsabfall
Copy link
Copy Markdown
Contributor

You're welcome 🎅

@Anteru Anteru added this to the 2.7.4 milestone Dec 24, 2020
@Anteru Anteru merged commit bd378d0 into pygments:master Dec 24, 2020
@Anteru
Copy link
Copy Markdown
Collaborator

Anteru commented Dec 24, 2020

Merged for the next release, thanks!

@Anteru Anteru added the changelog-update Items which need to get mentioned in the changelog label Dec 24, 2020
@Anteru Anteru self-assigned this Dec 24, 2020
@AkibAzmain
Copy link
Copy Markdown

@Anteru Can you please review my pull request #1642?

@Anteru Anteru removed the changelog-update Items which need to get mentioned in the changelog label Jan 6, 2021
@Anteru
Copy link
Copy Markdown
Collaborator

Anteru commented Jan 12, 2021

I just noticed while preparing the 2.7.4 release that the license terms don't match. I've reverted the change; can you please check again and commit it with the correct license? I presume this was all written by you and is original code, so it shouldn't be a problem. According to our Contributing guidelines, you must accept putting the code under 2-clause BSD.

Anteru added a commit that referenced this pull request Jan 12, 2021
This reverts commit bd378d0.

The license terms in dracula.py don't match. We'll add it back once the
author places it under the correct license.
@tomy0000000
Copy link
Copy Markdown

@danvilar Just in case you missed this, been waiting for this, please take a look. Thanks.

@danvilar
Copy link
Copy Markdown
Contributor Author

danvilar commented May 8, 2021

I will make a new pull request as soon as possible.

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.

5 participants