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
Fix various issues #13
Conversation
* Hammurabi only works in the current working directory * Read settings (pyproject.toml) path from ``HAMMURABI_SETTINGS_PATH`` environment variable * Fixed several CLI arguments related issues * Removed target directory setting from config and CLI
|
||
repo_url: str = self.repo.remote().url | ||
|
||
if re.match(r"^http(s)?://", repo_url): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think a simple repo_url.startswith('http://') or repo_url.startswith('https://')
is slightly more readable here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hm, maybe yes. The only concern I have against the startswith
is that it's longer, which could make the code less readable on the long run. WDYT?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am fine with the regex as well. 😃👍
Code Climate has analyzed commit ad1b26d and detected 0 issues on this pull request. The test coverage on the diff in this pull request is 100.0% (95% is the threshold). This pull request will bring the total coverage in the repository to 95.2% (0.2% change). View more on Code Climate. |
Reason for the change
We had various smaller issues across project. This PR will fix those issues and gives a bit better config handling.
Description
GitMixin
classHAMMURABI_SETTINGS_PATH
environment variableCode examples
N/A
Checklist
- [ ] Stubs created/updatedReferences
N/A