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

Update logrus from v0.8.7 to v1.0.5 #344

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

Conversation

alyssais
Copy link

@alyssais alyssais commented Jul 3, 2018

I need to report CodeClimate test results from a CI server running on SmartOS.

Since there's no official Solaris binary, I tried to build one myself. This didn't work, but after a while I found that all that was needed to get the build working was to update logrus to the latest version!

alyssais added 2 commits June 28, 2018 16:49
This will fix building on SmartOS.
@ale7714
Copy link
Contributor

ale7714 commented Jul 31, 2018

Hi @alyssais! Thank you for looking into this. Can you sign the CLA?

@alyssais
Copy link
Author

Done!

@alyssais
Copy link
Author

Hi! Noticed there's been some activity on this repo recently. Anything else I need to do before this can be merged?

@ale7714
Copy link
Contributor

ale7714 commented Jul 31, 2019

@alyssais sorry on the late follow up. Any chance you can rebase from master? I want to publish a beta binary and try this out in our CI for a day before releasing. Thank you!

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.

3 participants