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

Output git fetch/push to log files #2862

Merged
merged 4 commits into from
Oct 25, 2016
Merged

Output git fetch/push to log files #2862

merged 4 commits into from
Oct 25, 2016

Conversation

jacogr
Copy link
Contributor

@jacogr jacogr commented Oct 25, 2016

  • does not leak credentials when network is down
  • at this point removed the GitLab key until this is merged

@jacogr jacogr added A0-pleasereview 🤓 Pull request needs code review. M5-ui labels Oct 25, 2016
@jacogr jacogr added the F1-security 🛡 The client fails to follow expected, security-sensitive, behaviour. label Oct 25, 2016
@coveralls
Copy link

Coverage Status

Coverage increased (+0.003%) to 86.14% when pulling 779e38a on jg-bump-fix into 7eacf07 on master.

@gavofyork gavofyork added A2-insubstantial 👶 Pull request requires no code review (e.g., a sub-repository hash update). and removed A0-pleasereview 🤓 Pull request needs code review. F1-security 🛡 The client fails to follow expected, security-sensitive, behaviour. labels Oct 25, 2016
@gavofyork gavofyork merged commit e71c752 into master Oct 25, 2016
@gavofyork gavofyork deleted the jg-bump-fix branch October 25, 2016 15:39
@coveralls
Copy link

Coverage Status

Coverage remained the same at 86.136% when pulling 779e38a on jg-bump-fix into 7eacf07 on master.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
A2-insubstantial 👶 Pull request requires no code review (e.g., a sub-repository hash update).
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jacogr @coveralls @gavofyork