Skip to content
This repository was archived by the owner on Jul 19, 2025. It is now read-only.

Conversation

thatsanicehat
Copy link
Contributor

No description provided.

@calavera
Copy link
Contributor

calavera commented Jan 6, 2015

Hi @jv-dan, thanks for your contribution!

I understand that this solves your problem, but I'm not completely sure if it's the right way to fix this issue. You can see the discussion in #64

I'm totally open to suggestions though, I might be wrong and this might be the right solution for the future.

@thatsanicehat
Copy link
Contributor Author

Interesting. Looks like two separate, yet very related issues that may in fact be both solved by being able to specify the git root explicitly. My use-case was for being able to run tests and report on them when the ACTUAL, not a dummy app, doesn't live in the root of the git repository. I'll keep an eye on the discussion and see what others have to say!

@calavera
Copy link
Contributor

calavera commented Jan 6, 2015

Oh, thanks for clarifying it. That makes sense. I'm going to merge my changes, do you mind to rebase master with them in it? I'm sure you're going to have a small conflict. I can wait for you to release a new version of the gem, so we have both functionalities in the same release.

Conflicts:
	lib/code_climate/test_reporter/git.rb
@thatsanicehat
Copy link
Contributor Author

Sorry for holding up the release!

@calavera
Copy link
Contributor

calavera commented Jan 7, 2015

🆒 thanks @jv-dan. Merging!

calavera added a commit that referenced this pull request Jan 7, 2015
added support for changing the location of the .git root
@calavera calavera merged commit 383caac into codeclimate:master Jan 7, 2015
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.

2 participants