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

Implements changes made to the HAL a while ago for the Sim Notifier #87

Merged
merged 2 commits into from
Dec 21, 2015

Conversation

ThadHouse
Copy link
Member

Will make it easier to do a threaded notifier when that gets added to the official WPILib.

It changed in the HAL to move the process queuing down to the HAL. the
old process queue in the WPILib is still there, but it should be getting
updated soon to remove that.
@codecov-io
Copy link

Current coverage is 29.95%

Merging #87 into master will not affect coverage as of f75dd77

@@            master     #87   diff @@
======================================
  Files          123     123       
  Stmts         8123    8123       
  Branches      1029    1029       
  Methods          0       0       
======================================
  Hit           2433    2433       
  Partial        156     156       
  Missed        5534    5534       

Review entire Coverage Diff as of f75dd77

Powered by Codecov. Updated on successful CI builds.

ThadHouse added a commit that referenced this pull request Dec 21, 2015
Implements changes made to the HAL a while ago for the Sim Notifier
@ThadHouse ThadHouse merged commit 51e9a39 into master Dec 21, 2015
@ThadHouse ThadHouse deleted the NotifierChanges branch December 21, 2015 03:03
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.

2 participants