-
Notifications
You must be signed in to change notification settings - Fork 5
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
Release 0.5.2 #123
Release 0.5.2 #123
Conversation
Fix processing of sleep efficiency in FitbitSleepAvroConverter
Update routes to limit range to 30 days
Fix Fitbit routes and converter
Fix breathing and skin temperature convertors
Add Fitbit spo2 route and converter
Bump radar-schemas to latest version
Probably good to add a working codeql configuration, snyk workflow and perform security updates? I will make PRs. If there is a hurry you can make this release first but it shouldn't take long |
@Bdegraaf1234 Yes indeed would be good to have CodeQL working. Yes I can wait before making the release. Thank you! |
Add custom codeQL configuration
Why did the CodeQl action not run? 😆 Also maybe nice to rebase onto dev rather than merge to prevent unneeded merge commits |
@Bdegraaf1234 Ah yes will do. It didn't run because the branch was |
I think this one is ready to go after rebasing the security updates #126 ^^ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
17
, and other deps (Bump radar-schemas to latest version #122)