Skip to content

Standardize logging #48

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.

Sign up for GitHub

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

Open
diondokter wants to merge 6 commits into
base: mcxa-secure-boot
Choose a base branch
from
Open

Standardize logging #48

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
1285b79
Kick out the ROM api and use the HAL ROM api
diondokter Sep 7, 2026
d721014
Turn on defmt by default to make dev and CI easier
diondokter Sep 7, 2026
d1dfa66
Remove references to lifetime provisioning in CI
diondokter Sep 7, 2026
82b6eba
Add back licenses and make ci.sh work again
diondokter Sep 7, 2026
2500d09
Remove all custom logging macros
diondokter Sep 8, 2026
ef16959
Remove all references of the old feature flags
diondokter Sep 8, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
15 changes: 0 additions & 15 deletions .github/workflows/check.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -50,9 +50,6 @@ jobs:
- name: cargo fmt --check (bootloader-tool)
run: cargo fmt --check
working-directory: "./bootloader-tool"
- name: cargo fmt --check (mcxa-security-provisioning)
run: cargo fmt --check
working-directory: "./mcxa-security-provisioning"

clippy:
runs-on: ubuntu-latest
Expand Down Expand Up @@ -85,9 +82,6 @@ jobs:
- name: cargo clippy (bootloader-tool)
run: cargo clippy --locked -- -D clippy::suspicious -D clippy::correctness -D clippy::perf -D clippy::style
working-directory: "./bootloader-tool"
- name: cargo clippy (mcxa-security-provisioning)
run: cargo clippy --locked -- -F clippy::suspicious -D clippy::correctness -F clippy::perf -F clippy::style
working-directory: "./mcxa-security-provisioning"

# Enable once we have a released crate
# semver:
Expand Down Expand Up @@ -207,12 +201,6 @@ jobs:
log-level: warn
manifest-path: ./examples/mcxa-577app/Cargo.toml
command: check
- name: Cargo deny (mcxa-security-provisioning)
uses: EmbarkStudios/cargo-deny-action@v2
with:
log-level: warn
manifest-path: ./mcxa-security-provisioning/Cargo.toml
command: check

msrv:
# check that we can build using the minimal rust version that is specified by this crate
Expand Down Expand Up @@ -252,9 +240,6 @@ jobs:
- name: cargo +${{ matrix.msrv }} check (bootloader-tool)
run: cargo check --locked
working-directory: "./bootloader-tool"
- name: cargo +${{ matrix.msrv }} check (mcxa-security-provisioning)
run: cargo check
working-directory: "./mcxa-security-provisioning"

fuzz:
runs-on: ubuntu-latest
Expand Down
14 changes: 1 addition & 13 deletions .vscode/settings.json
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,22 +1,10 @@
{
"[markdown]": {
"editor.formatOnSave": false
},
"[toml]": {
"editor.formatOnSave": true
},
"[rust]": {
"editor.formatOnSave": true
},
"rust-analyzer.cargo.sysroot": "discover",
"rust-analyzer.check.allTargets": false,
"rust-analyzer.check.command": "clippy",
"rust-analyzer.cargo.target": "thumbv8m.main-none-eabihf",
"rust-analyzer.linkedProjects": [
"${workspaceFolder}/libs/Cargo.toml",
"${workspaceFolder}/examples/mcxa-577app/Cargo.toml",
],
"rust-analyzer.cargo.features": [
"mimxrt685s-evk",
"imxrt"
]
}
Loading
Loading