New issue

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

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

Already on GitHub? Sign in to your account

Blog section fixed #445

Merged
merged 7 commits into from Dec 11, 2016

Conversation

4 participants
@ankitrgadiya
Member

ankitrgadiya commented Dec 11, 2016

Do check Live preview
Issue resolved

@checkbot

This comment has been minimized.

checkbot commented Dec 11, 2016

1 Error
🚫 You’re modifying the index.html file. If you’re trying to add a student/blog/mentor, you’re doing it wrong. If you did this intentionally, please contact a mentor, as index.html is a core file.
2 Warnings
⚠️ Please provide a summary in the Pull Request description, containing, at less, your live link.
⚠️ Please add your live link or a preview screenshot to the PR body.
1 Message
📖 Hi @ankitrgadiya thank you for your submission at Fossasia’s GCI 2016 website. Please correct any issues above, if any.

Generated by 🚫 danger

@ankitrgadiya

This comment has been minimized.

Member

ankitrgadiya commented Dec 11, 2016

I have no idea why Travis checks are failing

@myfrom

This comment has been minimized.

Contributor

myfrom commented Dec 11, 2016

@ankitrgadiya You probably used tabs instead of spaces, that could cause Travis to fail

@ankitrgadiya

This comment has been minimized.

Member

ankitrgadiya commented Dec 11, 2016

My editor is configured to use spaces in place of tabs, probably because of previous merges

@jigyasa-grover jigyasa-grover merged commit d856aca into fossasia:gh-pages Dec 11, 2016

1 check failed

continuous-integration/travis-ci/pr The Travis CI build could not complete due to an error
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment