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

define undefined var dev5m #1418

Merged
merged 3 commits into from
Jan 5, 2022
Merged

define undefined var dev5m #1418

merged 3 commits into from
Jan 5, 2022

Conversation

scottleibrand
Copy link
Contributor

Fixes #1417: thx @Zanfib

@ChanceHarrison
Copy link

I applied the same fix locally and can confirm it works. Can we expect this to merge into dev soon?

@scottleibrand
Copy link
Contributor Author

That's sufficient testing for me, thx. Merging.

@scottleibrand scottleibrand merged commit 5e4ee98 into dev Jan 5, 2022
@scottleibrand scottleibrand deleted the var-dev5m branch January 5, 2022 01:31
bjornoleh pushed a commit to bjornoleh/oref0 that referenced this pull request Jan 9, 2022
* comment out shared-node cp to avoid full disk

* openaps#1417: define undefined var dev5m
bjornoleh added a commit to bjornoleh/oref0 that referenced this pull request Jan 9, 2022
bjornoleh pushed a commit to bjornoleh/oref0 that referenced this pull request Jan 12, 2022
* comment out shared-node cp to avoid full disk

* openaps#1417: define undefined var dev5m
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