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

ci: update linting toolchain to 1.53 #403

Merged
merged 2 commits into from Jul 19, 2021
Merged

Conversation

lucab
Copy link
Member

@lucab lucab commented Jul 15, 2021

This updates the linting toolchain to latest stable version and
fixes a couple of new warnings.

This updates the linting toolchain to latest stable version and
fixes a couple of new warnings.

Signed-off-by: Luca BRUNO <luca.bruno@coreos.com>
@lucab
Copy link
Member Author

lucab commented Jul 15, 2021

@breeswish it looks like Travis is gone. I'll work on porting the remaining jobs to github actions. In the meanwhile, can you please mark it as non-required to unblock this?

This removes the previous Travis configuration, moving all
remaining jobs into the existing GitHub workflow.

Signed-off-by: Luca BRUNO <luca.bruno@coreos.com>
@lucab
Copy link
Member Author

lucab commented Jul 15, 2021

I ended up folding the Travis migration into this PR.

Copy link
Contributor

@mxinden mxinden left a comment

Choose a reason for hiding this comment

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

Thanks for cleaning this up!

@BusyJay
Copy link
Member

BusyJay commented Jul 19, 2021

@zhouqiang-cl can you help cancel the requirement on travis ci?

@lucab
Copy link
Member Author

lucab commented Jul 19, 2021

Thanks all!

@lucab lucab merged commit bb90a19 into tikv:master Jul 19, 2021
@lucab lucab deleted the ups/ci-update-lints branch July 19, 2021 07:22
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.

None yet

3 participants