Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

add tests for twitter.card types #289

Merged
merged 1 commit into from
May 12, 2018

Conversation

set0gut1
Copy link
Contributor

I wrote test codes for twitter card type configuration (#225).
Please merge if these tests are beneficial.

I wrote tests because I could not change twitter card type of my github page which theme is choosed with "repository settings" webUI.
After wrote this, I noticed that #225 was merged only 9 days ago.

@DirtyF DirtyF added the tests label May 12, 2018
@DirtyF DirtyF requested a review from a team May 12, 2018 09:52
Copy link
Member

@DirtyF DirtyF left a comment

Choose a reason for hiding this comment

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

Thanks @set0gut1 for adding those tests

@set0gut1
Copy link
Contributor Author

Thank you too ✨

@DirtyF
Copy link
Member

DirtyF commented May 12, 2018

@jekyllbot: merge +dev

@jekyllbot jekyllbot merged commit f8800e5 into jekyll:master May 12, 2018
jekyllbot added a commit that referenced this pull request May 12, 2018
@set0gut1 set0gut1 deleted the add-test-twitter-card branch May 13, 2018 04:17
@jekyll jekyll locked and limited conversation to collaborators May 13, 2019
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants