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

OLS cleanups #84

Closed
wants to merge 8 commits into from
Closed

OLS cleanups #84

wants to merge 8 commits into from

Conversation

v1ne
Copy link

@v1ne v1ne commented Aug 5, 2020

These clean-ups are taken from #51 and are the base to further refactor the driver and add support for advanced features of the DemonCore.

@v1ne v1ne mentioned this pull request Aug 5, 2020
Copy link

@wsakernel wsakernel left a comment

Choose a reason for hiding this comment

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

All my comments from PR #51 have been addressed. This subset is ready to go IMO.

@gsigh gsigh self-assigned this Aug 23, 2020
@gsigh
Copy link
Contributor

gsigh commented Aug 23, 2020

I can pick up these improvements. @wsakernel shall I add your Acked-By?

@wsakernel
Copy link

Technically, it would be more Reviewed-by. But I don't mind much which one... Thanks!

@gsigh
Copy link
Contributor

gsigh commented Aug 24, 2020

Merged in 6f9234e, thank you. Have added wsa's Reviewed-By. Had to adjust the commit messages. Would be nice if your future commits would stick with the project's habit of prefixing the subject with the driver name. Makes it easier to navigate later during research.

@gsigh gsigh closed this Aug 24, 2020
@v1ne
Copy link
Author

v1ne commented Aug 27, 2020

@gsigh: Sure, will do! Thanks for telling me.

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.

3 participants