[MegaLinter] Apply linters automatic fixes #50
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ScottGibb
force-pushed
the
megalinter-fixes-refactor-ci-tests
branch
from
016faa4 to
c570ae8
Compare
ScottGibb
force-pushed
the
megalinter-fixes-refactor-ci-tests
branch
from
c570ae8 to
70faee2
Compare
ScottGibb
added a commit
that referenced
this pull request
Jan 10, 2025
* cold start tests are now generic * setting register tests now generic * updated CI to use Linux Tests * fixed markdown lint issue * refactored into test crate * updated CI and tests documentation * restructured tests and now use common folder. Paths work and can call tests from the top folder * added breaking changes. Cant seem to get the i2c shared bus to work investigating changing traits * reverted change * fixed bus mutex example * fixed readme links * added first stab at esp test harness * Attempting to pass i2c * rearranging dependencies * cargo test runs but cant link rust-lld * Start from scratch * removed old workspace * test framework works using cargo test inside the esp32c3 tests folder * removed unecessary log feature * removed extra dependancies * doc changes * added first esp32 i2c test * removed unused aliases * restructured tests to make reusable tests * Seperated workspaces as the MCU environemnts might have different lock files * removed them from library workspace * removed unecessary noise * Updated docs * updated examples * cold_start_tests confirmed working * added setting_register_tests * added CI updated tests * Fixed broken links * updated lint checks for new tests directory * moved esp32c3 example * fixed workflows * fixed rust analyser * Fixed Links * addressing clippy warning * Updated docs * added probe-rs to pipeline * [MegaLinter] Apply linters automatic fixes (#49) * fixing install * Added missing dependencies * removing bininstall * debugging * adding probe-rs * Added back in tests * [MegaLinter] Apply linters automatic fixes (#50) * fixed apt in workflow and changed imports back to original * fixed mut and imports * fixed public * added probe specifics * added probe reset * updating probe-rs install * fixing cmd * cahnging runner to use debian instead of ubuntu * swapping to bookworm * tagging bookworm docker version * tagged version * added missing installs * Splitting tests up * adding missing source * correcting order * disabled faulty test * fixed comment string * removed duplicated project * added config variable * fixed double quoting * formatting * added short readme * updated broken link * reduced boiler plate
ScottGibb
added a commit
that referenced
this pull request
Jan 13, 2025
* cold start tests are now generic * setting register tests now generic * updated CI to use Linux Tests * fixed markdown lint issue * refactored into test crate * updated CI and tests documentation * restructured tests and now use common folder. Paths work and can call tests from the top folder * added breaking changes. Cant seem to get the i2c shared bus to work investigating changing traits * reverted change * fixed bus mutex example * fixed readme links * added first stab at esp test harness * Attempting to pass i2c * rearranging dependencies * cargo test runs but cant link rust-lld * Start from scratch * removed old workspace * test framework works using cargo test inside the esp32c3 tests folder * removed unecessary log feature * removed extra dependancies * doc changes * added first esp32 i2c test * removed unused aliases * restructured tests to make reusable tests * Seperated workspaces as the MCU environemnts might have different lock files * removed them from library workspace * removed unecessary noise * Updated docs * updated examples * cold_start_tests confirmed working * added setting_register_tests * added CI updated tests * Fixed broken links * updated lint checks for new tests directory * moved esp32c3 example * fixed workflows * fixed rust analyser * Fixed Links * addressing clippy warning * Updated docs * added probe-rs to pipeline * [MegaLinter] Apply linters automatic fixes (#49) * fixing install * Added missing dependencies * removing bininstall * debugging * adding probe-rs * Added back in tests * [MegaLinter] Apply linters automatic fixes (#50) * fixed apt in workflow and changed imports back to original * fixed mut and imports * fixed public * added probe specifics * added probe reset * updating probe-rs install * fixing cmd * cahnging runner to use debian instead of ubuntu * swapping to bookworm * tagging bookworm docker version * tagged version * added missing installs * Splitting tests up * adding missing source * correcting order * disabled faulty test * fixed comment string * removed duplicated project * added config variable * fixed double quoting * formatting * Attempt at adding rp pico * cant seem to clear up i2c generics * added working rp pico example!! * tested and runs * updated workflow * update cargo dependencies * added missing toolchain * added short readme * updating structure and readme docs * Added docs updates * clippy issues and adding it to the CI * fixing names * Fixed names * investigate this * getting rid of probe resets * updated broken link * fixed rust analyser issue * minor tidying * further tidy up * removed todos * fixed docs * add space * spelling fix * added H * removing import * Bug Fix for naming * reducing boiler plate * fixing clippy issue * reduced boiler plate * adding target for clippy * making clippy happy * fixed clippy warning * agreed removed :) * fixed extra # * removed unecessary comments
ScottGibb
added a commit
that referenced
this pull request
Jan 14, 2025
* cold start tests are now generic * setting register tests now generic * updated CI to use Linux Tests * fixed markdown lint issue * refactored into test crate * updated CI and tests documentation * restructured tests and now use common folder. Paths work and can call tests from the top folder * added breaking changes. Cant seem to get the i2c shared bus to work investigating changing traits * reverted change * fixed bus mutex example * fixed readme links * added first stab at esp test harness * Attempting to pass i2c * rearranging dependencies * cargo test runs but cant link rust-lld * Start from scratch * removed old workspace * test framework works using cargo test inside the esp32c3 tests folder * removed unecessary log feature * removed extra dependancies * doc changes * added first esp32 i2c test * removed unused aliases * restructured tests to make reusable tests * Seperated workspaces as the MCU environemnts might have different lock files * removed them from library workspace * removed unecessary noise * Updated docs * updated examples * cold_start_tests confirmed working * added setting_register_tests * added CI updated tests * Fixed broken links * updated lint checks for new tests directory * moved esp32c3 example * fixed workflows * fixed rust analyser * Fixed Links * addressing clippy warning * Updated docs * added probe-rs to pipeline * [MegaLinter] Apply linters automatic fixes (#49) * fixing install * Added missing dependencies * removing bininstall * debugging * adding probe-rs * Added back in tests * [MegaLinter] Apply linters automatic fixes (#50) * fixed apt in workflow and changed imports back to original * fixed mut and imports * fixed public * added probe specifics * added probe reset * updating probe-rs install * fixing cmd * cahnging runner to use debian instead of ubuntu * swapping to bookworm * tagging bookworm docker version * tagged version * added missing installs * Splitting tests up * adding missing source * correcting order * disabled faulty test * fixed comment string * removed duplicated project * added config variable * fixed double quoting * formatting * Attempt at adding rp pico * cant seem to clear up i2c generics * added working rp pico example!! * tested and runs * updated workflow * update cargo dependencies * added missing toolchain * added short readme * updating structure and readme docs * Added docs updates * clippy issues and adding it to the CI * fixing names * Fixed names * investigate this * getting rid of probe resets * updated broken link * fixed rust analyser issue * minor tidying * further tidy up * removed todos * flashes with probe-rs * Linker Script update. Code now flashes * I2c traits are incompatible * fixed docs * add space * spelling fix * added H * removing import * code works however embedded-test pulls in std for some reason? Which then shoots the binary size for the tests. Will have to split them up and duplicate * Bug Fix for naming * split up all tests they all run and pass now * added linter and ci stages for stm tests * [MegaLinter] Apply linters automatic fixes (#55) * updating docs * removed broken feature * reduce boiler plate and doc * reducing boiler plate * fixing clippy issue * reduced boiler plate * adding target for clippy * added rustup targets * updated * making clippy happy * fixed clippy warning * removed dead code * adding check gaurds and swapping digi key link * agreed removed :) * fixed extra # * removed unecessary comments * passing lint checks * fixed cargo.toml white space * removed old dependencies * make comment more clear * docs fixes * adding dev optimisation settings
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
MegaLinter has automatically applied linters fixes on this PR.
Please review the changes and merge if they are correct.