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

v0.18.0 🎉 #425

Merged
merged 60 commits into from
Jan 14, 2025
Merged

v0.18.0 🎉 #425

merged 60 commits into from
Jan 14, 2025

Conversation

simonmicro
Copy link
Member

@simonmicro simonmicro commented Jan 11, 2025

After #423, #424

@simonmicro simonmicro added 🐞 bug Something isn't working ⭐ enhancement New feature or request src/osw-app This PR / issue impacts the current release of the OS src/core src/osw-service 📜 documentation This adds / changes the documentation 🎉 release This PR / issue impacts the current release of the OS connection/Bluetooth labels Jan 11, 2025
@simonmicro simonmicro self-assigned this Jan 11, 2025
@simonmicro simonmicro added this to the v0.18.0 milestone Jan 11, 2025
@simonmicro simonmicro marked this pull request as ready for review January 12, 2025 20:54
@RuffaloLavoisier RuffaloLavoisier self-requested a review January 12, 2025 22:25
src/services/OswServiceTaskConsole.cpp Show resolved Hide resolved
src/osw_config_keys.cpp Show resolved Hide resolved
include/apps/watchfaces/OswAppWatchfaceDigital.h Outdated Show resolved Hide resolved
include/services/OswServiceTaskBLEServer.h Show resolved Hide resolved
include/services/OswServiceTaskBLEServer.h Show resolved Hide resolved
include/services/OswServiceTaskBLEServer.h Show resolved Hide resolved
include/services/OswServiceTaskBLEServer.h Show resolved Hide resolved
include/services/OswServiceTaskBLEServer.h Show resolved Hide resolved
Signed-off-by: simonmicro <[email protected]>
src/apps/watchfaces/OswAppWatchfaceFitnessAnalog.cpp Outdated Show resolved Hide resolved
src/apps/watchfaces/OswAppWatchfaceDigital.cpp Outdated Show resolved Hide resolved
src/apps/watchfaces/OswAppWatchfaceFitnessAnalog.cpp Outdated Show resolved Hide resolved
src/apps/watchfaces/OswAppWatchfaceDigital.cpp Outdated Show resolved Hide resolved
@simonmicro
Copy link
Member Author

simonmicro commented Jan 14, 2025

Good point. I'll take a look into it later. But, for next time, shouldn't we catch and discuss this earlier? Like on the PR initially introducing this and then failing to integrate? Sorry, a little frustrated, because I wanted to merge this Monday.

- Move variables that are compiled but may not be used due to macros into the scope of the macros.
@simonmicro
Copy link
Member Author

simonmicro commented Jan 14, 2025

Sorry for the force - I feel like its Monday again... 😑

Copy link
Member

@RuffaloLavoisier RuffaloLavoisier left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@simonmicro simonmicro added the ✔️ merge when CI passed Note for the maintainers: Merge the PR ASAP after the CI finished label Jan 14, 2025
@RuffaloLavoisier RuffaloLavoisier merged commit cf10dd6 into master Jan 14, 2025
243 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
connection/Bluetooth ✔️ merge when CI passed Note for the maintainers: Merge the PR ASAP after the CI finished 🐞 bug Something isn't working 📜 documentation This adds / changes the documentation src/core src/osw-app This PR / issue impacts the current release of the OS src/osw-service ⭐ enhancement New feature or request 🎉 release This PR / issue impacts the current release of the OS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants