Skip to content
This repository was archived by the owner on Aug 18, 2020. It is now read-only.

Removal of hidden dependency to RTCZero which is used to retrieve a epoch type timestamp #56

Merged
merged 1 commit into from
Mar 9, 2020

Conversation

aentinger
Copy link
Contributor

In #55 the possibility for retrieving a epoch type timestamp via a registered callback function has been introduced, the changes in arduino-libraries/ArduinoIoTCloud#93 register such a function which allows us to to remove the hidden dependency to RTCZero altogether.

@TravisBuddy

This comment has been minimized.

…poch type timestamp

In #55 the possibility for retrieving a epoch type timestamp via a registered callback function has been introduced, the changes in arduino-libraries/ArduinoIoTCloud#93 register such a function which allows us to to remove the hidden dependency to RTCZero altogether
@aentinger aentinger force-pushed the cleanup-remove-extern-RTCZero-dependency branch from 6083e59 to bcd9374 Compare February 27, 2020 06:13
@codecov-io
Copy link

codecov-io commented Feb 27, 2020

Codecov Report

Merging #56 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #56      +/-   ##
==========================================
- Coverage   95.41%   95.39%   -0.02%     
==========================================
  Files          24       24              
  Lines         872      869       -3     
==========================================
- Hits          832      829       -3     
  Misses         40       40
Impacted Files Coverage Δ
src/ArduinoCloudProperty.cpp 93.61% <ø> (-0.14%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9c8aa23...bcd9374. Read the comment docs.

@aentinger aentinger marked this pull request as ready for review March 6, 2020 09:56
@aentinger
Copy link
Contributor Author

Successfully tested on PROD with

  • MKR WiFi 1010 ✔️
  • MKR GSM 1400 ✔️
  • ESP 8266 ✔️
  • MKR WAN 1310 ✔️

@aentinger aentinger merged commit fb94b6d into master Mar 9, 2020
@aentinger aentinger deleted the cleanup-remove-extern-RTCZero-dependency branch March 9, 2020 10:04
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.

4 participants