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

Refactor CI config #120

Merged
merged 1 commit into from
Apr 15, 2019
Merged

Refactor CI config #120

merged 1 commit into from
Apr 15, 2019

Conversation

dennwc
Copy link
Member

@dennwc dennwc commented Apr 12, 2019

Refactor CI config as per @bzz suggestion. See this thread for details.

Signed-off-by: Denys Smirnov denys@sourced.tech

@dennwc dennwc requested a review from bzz April 12, 2019 13:34
@dennwc dennwc self-assigned this Apr 12, 2019
@dennwc dennwc mentioned this pull request Apr 12, 2019
Merged
ci: refactor config, remove unused import path
2abfa8d 
Signed-off-by: Denys Smirnov <denys@sourced.tech>
@dennwc
Copy link
Member Author

dennwc commented Apr 12, 2019

@bzz Works well in the fork. I think we can merge it safely.

bzz
bzz approved these changes Apr 14, 2019
View reviewed changes
Copy link
Contributor

@bzz bzz left a comment

Choose a reason for hiding this comment

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

🚀

@dennwc dennwc merged commit 90c2185 into bblfsh:master Apr 15, 2019
@dennwc dennwc deleted the ci_cleanup branch April 15, 2019 11:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@bzz bzz bzz approved these changes

Assignees

@dennwc dennwc

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dennwc @bzz