Skip to content
This repository has been archived by the owner on Oct 22, 2019. It is now read-only.

fixed read sync bug #8

Merged
merged 2 commits into from
Oct 8, 2019
Merged

fixed read sync bug #8

merged 2 commits into from
Oct 8, 2019

Conversation

TimonPost
Copy link
Member

No description provided.

Copy link
Contributor

@zrzka zrzka left a comment

Choose a reason for hiding this comment

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

LGTM. Just add git & branch master dependency (you can keep version there).

@TimonPost TimonPost merged commit bb53032 into master Oct 8, 2019
@zrzka
Copy link
Contributor

zrzka commented Oct 8, 2019

I know that you don't like Travis CI speed, but please don't merge before it's green. We should set it as required check (merge button is disabled till green) :)

@TimonPost
Copy link
Member Author

TimonPost commented Oct 8, 2019

Actually, I had a reason why I did that, but that reason turns out to be invalid. I was a bit to fast and clicked merge to fast without validating that thought. Was still used to the old scenario where everything was one repo

@TimonPost TimonPost deleted the update_winapi branch October 11, 2019 12:39
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.

2 participants