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

[check-log] Jsonize status file #252

Merged
merged 3 commits into from Sep 12, 2018

Conversation

Projects
None yet
2 participants
@yuuki
Member

yuuki commented Sep 7, 2018

It is for future extensions such as #250.
This PR supports backward compatibility for old style of status file.

yuuki added some commits Sep 7, 2018

Show outdated Hide outdated check-log/lib/check-log.go Outdated
@yuuki

This comment has been minimized.

Show comment
Hide comment
@yuuki

yuuki Sep 12, 2018

Member

@astj Do you have any other comment?

Member

yuuki commented Sep 12, 2018

@astj Do you have any other comment?

@astj

astj approved these changes Sep 12, 2018

@astj

This comment has been minimized.

Show comment
Hide comment
@astj

astj Sep 12, 2018

Contributor

@yuuki Nothing. LGTM!

Contributor

astj commented Sep 12, 2018

@yuuki Nothing. LGTM!

@yuuki

This comment has been minimized.

Show comment
Hide comment
@yuuki

yuuki Sep 12, 2018

Member

Thanks!

Member

yuuki commented Sep 12, 2018

Thanks!

@yuuki yuuki merged commit 488c46a into master Sep 12, 2018

3 checks passed

continuous-integration/appveyor/branch AppVeyor build succeeded
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
coverage/coveralls Coverage increased (+0.02%) to 55.705%
Details

@yuuki yuuki deleted the check-log-jsonize branch Sep 12, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment