UncleRus / esp-idf-lib

Component library for ESP32-xx and ESP8266
https://esp-idf-lib.readthedocs.io/en/latest/
1.35k stars 420 forks source link

feature: add LSM303x accelerometer/magnetometer support #571

Closed QB4-dev closed 11 months ago

QB4-dev commented 11 months ago

My LSM303 accelerometer/magnetometer sensor driver implementation

Feature requested by issue https://github.com/UncleRus/esp-idf-lib/issues/570

UncleRus commented 11 months ago

In the current state, this driver cannot be merged into the master: there are practically no doxygen comments in the sources, so there will be no documentation. Please place comments in the header file for all functions as it is done in other drivers.

UncleRus commented 11 months ago

Also, I would like to ask you not to use tabs, only spaces. In any case, you can always clang-format your files, the repository contains the correct settings.

QB4-dev commented 11 months ago

Sorry about missing doxygen comments, now it is fixed.

I also changed my editor settings to no tabs, and IndentWidth: 4 Now should be OK

UncleRus commented 11 months ago

Thank you!