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

Update url of schema website #296

Merged
merged 1 commit into from
May 23, 2018
Merged

Update url of schema website #296

merged 1 commit into from
May 23, 2018

Conversation

alokpant
Copy link
Contributor

Change it from http to https. Details of issue is
at #294

Change it from http to https. Details of issue is
at #294
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 @alokpant

@DirtyF
Copy link
Member

DirtyF commented May 22, 2018

@alokpant Rubocop issue is unrelated to this PR, we'll deal with it later.

@alokpant
Copy link
Contributor Author

Thank you @DirtyF for approving.

Copy link
Collaborator

@benbalter benbalter left a comment

Choose a reason for hiding this comment

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

🎉

@benbalter
Copy link
Collaborator

Congrats on your first pull request to Jekyll SEO Tag! Without amazing humans like you submitting pull requests, we couldn’t run this project. You rock! 🎉

If you're interested in tackling another bug or feature, take a look at the open issues, especially those labeled help wanted.

@alokpant
Copy link
Contributor Author

Thank you @benbalter. Will definitely look more to see if I can contribute.

@DirtyF
Copy link
Member

DirtyF commented May 23, 2018

@jekyllbot: merge +fix

@jekyllbot jekyllbot merged commit a50f80b into jekyll:master May 23, 2018
jekyllbot added a commit that referenced this pull request May 23, 2018
@alokpant alokpant deleted the issue/schema_url branch May 23, 2018 21:31
@jekyll jekyll locked and limited conversation to collaborators May 23, 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