Activity since 2025-05-29

• esp-rs/esp-generate repo issues and pull requests

  • Fix RTT without defmt #183 [bugadani]

    rtt_init by default initializes 0 channels, which causes probe-rs to crash currently, and isn't useful besides.

    • bjoernQ: I think rtt_target::rtt_init_print!(); isn't emitted if the user doesn't use panic-rtt-target...

    • bjoernQ: Thanks

  • Add trouble #179 [jamessizeland]

    Here's a first pass at adding a trouble example to the esp-generate template. Very nice work on that templating...

    • jamessizeland: Do you need anything else from me on this PR?

    • SergioGasquez: LGTM! Maybe Id rename the ble option to ble-bleps to be consistent with the naming, as someone...

  • Clean up #184 [bugadani]

    • bjoernQ: 👍

  • Test each option for each base template, fix build errors, pin esp-hal #186 [bugadani] skip-changelog

    Closes #185

    • bjoernQ: LGTM

  • Add esp-idf-bootloader support #165 [MabezDev]

    We will need to start generating projects with the new esp-idf app desc macro here, once the new espflash release is out.

  • Add esp-bootloader-esp-idf support #166 [JurajSadel]

    closes #165

    • bjoernQ: Could we find a slightly more helpful comment?

    • bugadani: Suggested change // This creates a...

    • bjoernQ: the better link would be:...

    • SergioGasquez: LGTM! Are we checking that the project builds with trouble? We may need to patch it to a git rev

    • bjoernQ: just out of curiosity: why do we use log as log-04 in a (mostly) binary crate?

      • JurajSadel: 👍

    • bjoernQ: this should fix #185 - maybe the expect is a little bit nicer than a plain unwrap?

    • bjoernQ: LGTM - Thanks! Before publishing we probably want to run a full check

    • bjoernQ: Not exactly needed but we can update this: esp-generate/Cargo.toml ...

    • bugadani: LGTM, but you should probably add at least the esp-hal update to the changelog

    • MabezDev

      • MabezDev: Let's include a link to the esp-idf docs about the app desc.

      • MabezDev: Suggested change // This creates a...

created at 2025-06-05 02:01 by dinghy 0.13.1.