Skip to content
This repository has been archived by the owner on Sep 9, 2020. It is now read-only.

Allow git-lint to check branches #81

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

Conversation

dougalg
Copy link

@dougalg dougalg commented Jun 11, 2016

This modifies the behaviour (for Git only) to accept --branch specifier which will allow users to get linting diffs between branches.Useful for testing after commits and PRs have been made. This also now gives the ability (for git) to test arbitrary commit distances apart (although that UI is not implemented here)

I've had some trouble running the tests, but I think I'm getting closer to having them fixed up soon. Also, I don't consider this complete, but would like to get feedback on what you think of my approach here before I go any further.

Thanks for the great tool and hopefully you find this concept useful!

@dougalg dougalg force-pushed the branch-diff branch 4 times, most recently from a147130 to e9e278c Compare June 12, 2016 19:30
@dougalg
Copy link
Author

dougalg commented Jun 12, 2016

Looks like there's still a couple of failures in the end to end tests. Not sure how to resolve those. @sk- if you can provide any insight?

This modified the behaviour (for Git only) to accept --branch specifier which will allow users to get linting diffs between branches.Useful for testing after commits and PRs have been made. This also now gives the ability (for git) to test arbitrary commit distances apart (although that UI is not implemented here)
@atourkow
Copy link

atourkow commented Mar 7, 2018

@sk- Would like to see this added. Would be super useful to check against PRs in Travis builds.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants