Skip to content
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

Add GitHub ribbons #225

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

siddhpant
Copy link
Member

Closes #24.

Used http://tholman.com/github-corners/ as it's used in some other coala repos and was also suggested in the issue discussion.

@TravisBuddy

This comment has been minimized.

templates/ribbon.html Outdated Show resolved Hide resolved
@TravisBuddy

This comment has been minimized.

@TravisBuddy

This comment has been minimized.

templates/ribbon.html Outdated Show resolved Hide resolved
@TravisBuddy

This comment has been minimized.

templates/model/templates/ribbon.html Outdated Show resolved Hide resolved
templates/ribbon.html Show resolved Hide resolved
templates/ribbon.html Outdated Show resolved Hide resolved
@TravisBuddy

This comment has been minimized.

Copy link

@SerekKiri SerekKiri left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@jayvdb
Copy link
Member

jayvdb commented Nov 29, 2018

@gitmate-bot rebase

@gitmate-bot
Copy link
Collaborator

Hey! I'm GitMate.io! This pull request is being rebased automatically. Please DO NOT push while rebase is in progress or your changes would be lost permanently ⚠️

@gitmate-bot
Copy link
Collaborator

Automated rebase with GitMate.io was successful! 🎉

templates/ribbon.html Outdated Show resolved Hide resolved
@TravisBuddy

This comment has been minimized.

@TravisBuddy

This comment has been minimized.

@siddhpant siddhpant changed the title WIP: Add GitHub ribbons Add GitHub ribbons Dec 4, 2018
templates/ribbon.html Outdated Show resolved Hide resolved
@TravisBuddy

This comment has been minimized.

Copy link
Member

@li-boxuan li-boxuan left a comment

Choose a reason for hiding this comment

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

@siddhpant
Copy link
Member Author

Done

Copy link
Member

@li-boxuan li-boxuan left a comment

Choose a reason for hiding this comment

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

Almost there 👍

community/settings.py Outdated Show resolved Hide resolved
Copy link
Member

@li-boxuan li-boxuan left a comment

Choose a reason for hiding this comment

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

LGTM 👍

data/templatetags/ribbon_link.py Outdated Show resolved Hide resolved
@TravisBuddy
Copy link

Travis tests have failed

Hey @siddhpant,
Please read the following log in order to understand the failure reason.
It'll be awesome if you fix what's wrong and commit the changes.

1st Build

View build log

coala --non-interactive -V
[ERROR][07:14:47] There is a conflict in the version of a dependency you have installed and the requirements of coala. This may be resolved by creating a separate virtual environment for coala or running `pip3 install "git-url-parse~=1.1.0"`. Be aware that the latter solution might break other python packages that depend on the currently installed version.
[INFO][07:14:47] Exception was:
Traceback (most recent call last):

  File "/home/travis/virtualenv/python3.6.3/lib/python3.6/site-packages/coalib/coala_main.py", line 147, in run_coala
    args=args)

  File "/home/travis/virtualenv/python3.6.3/lib/python3.6/site-packages/coalib/settings/ConfigurationGathering.py", line 504, in gather_configuration
    targets=targets,

  File "/home/travis/virtualenv/python3.6.3/lib/python3.6/site-packages/coalib/settings/SectionFilling.py", line 97, in fill_settings
    bear_dirs = section.bear_dirs()

  File "/home/travis/virtualenv/python3.6.3/lib/python3.6/site-packages/coalib/settings/Section.py", line 147, in bear_dirs
    for bear_dir in collect_registered_bears_dirs('coalabears')]

  File "/home/travis/virtualenv/python3.6.3/lib/python3.6/site-packages/coalib/collecting/Collectors.py", line 456, in collect_registered_bears_dirs
    registered_package = ep.load()

  File "/home/travis/virtualenv/python3.6.3/lib/python3.6/site-packages/pkg_resources/__init__.py", line 2404, in load
    self.require(*args, **kwargs)

  File "/home/travis/virtualenv/python3.6.3/lib/python3.6/site-packages/pkg_resources/__init__.py", line 2427, in require
    items = working_set.resolve(reqs, env, installer, extras=self.extras)

  File "/home/travis/virtualenv/python3.6.3/lib/python3.6/site-packages/pkg_resources/__init__.py", line 875, in resolve
    raise VersionConflict(dist, req).with_context(dependent_req)

pkg_resources.VersionConflict: (git-url-parse 1.2.0 (/home/travis/virtualenv/python3.6.3/lib/python3.6/site-packages), Requirement.parse('git-url-parse~=1.1.0'))
TravisBuddy Request Identifier: 685c3730-2c3a-11e9-b3d3-256864349aa7

@siddhpant
Copy link
Member Author

siddhpant commented Feb 9, 2019

Travis fails due to new version of git-url-parse being available (1.2.0)

#237

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

8 participants