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

fixes #381 #395

Closed
wants to merge 7 commits into from
Closed

fixes #381 #395

wants to merge 7 commits into from

Conversation

ktpv
Copy link
Contributor

@ktpv ktpv commented Nov 24, 2019

Signed-off-by: ktpv ktpv@users.noreply.github.com

jromero and others added 3 commits November 20, 2019 06:42
Reason: Other implementations satisfy the purpose of the fake logger. It's thereby
simply more code to maintain without added value

Signed-off-by: Javier Romero <jromero@pivotal.io>
Signed-off-by: Javier Romero <jromero@pivotal.io>
Signed-off-by: ktpv <ktpv@users.noreply.github.com>
@ktpv ktpv requested a review from a team as a code owner November 24, 2019 16:25
Copy link
Member

@jromero jromero left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi 👋, thanks for the contribution!

There are two lingering issues with this PR which prevent us from merging it:

  1. The tests are not passing on travis.
  2. The diff appears to show duplicate commits. Looking at the commit tree confirms the duplication.
*   2019-12-03 36576af - (HEAD -> ktpv-i381) Merge branch 'i381' of https://github.com/ktpv/pack into i381 [ktpv] N
|\  
| * 2019-11-24 b93fdab - fixes #381 [ktpv] N
* | 2019-11-24 d77e904 - fixes #381 [ktpv] N
* |   2019-12-01 789347b - (origin/master, origin/HEAD) Merge pull request #391 from buildpack/feature/moar-badges [GitHub] E

You might find some information available in our CONTRIBUTING.md helpful.

@ktpv
Copy link
Contributor Author

ktpv commented Dec 4, 2019

Redid in PR #404. Closing this out.

@ktpv ktpv closed this Dec 4, 2019
@ktpv ktpv deleted the i381 branch December 4, 2019 16:05
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