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

[logs] Add new log messages #137

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

[logs] Add new log messages #137

wants to merge 2 commits into from

Conversation

mafesan
Copy link
Contributor

@mafesan mafesan commented Apr 21, 2017

After a first revision of the Git backend module, a few new log messages has been added:

  • It may be useful to distinguish between parsing a given git log file or it has to obtain that log file from a repository.

logger.info("Fetching commits from the given log file %s", self.gitpath)
logger.info("Log file not specified, retrieving from repository.")

  • As proposed, now a message informs if the repository is found locally or it has to be cloned first.

logger.info("Git repository not found in %s", self.gitpath)
logger.info("Found Git repository in %s", self.gitpath)

  • Message logger.debug("Git %s repository cloned into %s",... has been changed into info type, as this information may help to discriminate if the program has stuck in the cloning phase.

Miguel Angel Fernandez added 2 commits April 21, 2017 11:11
New log messages: inform if a git log file has been specified or not, and inform if the repository has been found locally or it  has to be cloned
@coveralls
Copy link

Coverage Status

Coverage increased (+0.005%) to 95.736% when pulling b6b6938 on mghfdez:logs into ccc317c on grimoirelab:master.

@jgbarah
Copy link
Contributor

jgbarah commented Aug 5, 2017

I'm sorry this patch is now conflicting. I'm sorry this was sitting here for so long, but if you still think it is relevant, would you mind updtaing it to the latest version in the master branch?

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

Successfully merging this pull request may close these issues.

3 participants