Bump third_party/pigweed/repo from 2e14574 to 8da9f28#38
Closed
dependabot[bot] wants to merge 1 commit intomasterfrom
Closed
Bump third_party/pigweed/repo from 2e14574 to 8da9f28#38dependabot[bot] wants to merge 1 commit intomasterfrom
2e14574 to 8da9f28#38dependabot[bot] wants to merge 1 commit intomasterfrom
Conversation
Bumps [third_party/pigweed/repo](https://github.com/google/pigweed) from `2e14574` to `8da9f28`. - [Release notes](https://github.com/google/pigweed/releases) - [Commits](pigweed-project/pigweed@2e14574...8da9f28) --- updated-dependencies: - dependency-name: third_party/pigweed/repo dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com>
Author
|
Superseded by #39. |
dependabot
bot
deleted the
dependabot/submodules/third_party/pigweed/repo-8da9f28
branch
LuDuda
pushed a commit
that referenced
this pull request
Jul 30, 2025
* Implement partial solution up to step 5 * Draft: Work in progress until step 22 * Draft PR - Verify assert values and finalize test case for TC_CGEN_2_2 * Improvements in failsafe handling, timeouts, and preparation for step 44 in TC_CGEN_2_2 * Implemented failsafe expiry workaround, cleanup for TC_CGEN_2_2, and step 44 updated * Updated code-lints and fixed f-string warnings * Fixed import order and to conflicts for Restyled * Code review done, changes and refactoring implemented * Improving long waits for cert test * Add PICS argument for CI * Refactor code: improved flow handling and optimized wait times * Refactor function call to is_pics_sdk_ci_only and update/improve waits * Fixed: Lint Code Base * Code cleanup and restyle * Fixed: Update try/except with specific exceptions and fail the test if the expected exception is not raised * Fixed: Rename fn to for clarity based on feedback * Fixed: Removed UTC time function, logging elapsed time in ms in steps 41 and 43. * Fixed: Hooked failsafe_expiration_seconds to user parameter PIXIT.CGEN.FailsafeExpiryLengthSeconds for CI or Cert flow flexibility. * Fixed: Lint Code base and Restyled. * Fixed: Argument PIXIT.CGEN.FailsafeExpiryLengthSeconds updated to 1 * Added constant variable FAILSAFE_EXPIRATION_SECONDS for cert tests to handle timeout more effectively * Add step 2a to configure failsafe timer for CI and Cert environments, and create constant RUN_TEST_ to define the run test type. * Fixed: Adjust failsafe timer for CI and Cert, with configurable expiration and default value of 20 seconds * Fixed: Step 7 updated. * Fixed: Step 2a updated in order to match with test plan. * Fixes and updates from feedback * Fixed: Step #26 logger removed. * Fixed: Bypass steps from #38 onward ONLY in CI. * Fixed: Set expiration_time_seconds=1 in Step #7, and minor adjustments to PIXIT.CGEN.FailsafeExpiryLengthSeconds. * Re-use root certificate from step 5 in steps 16, 27, 30, and 39 * Fixed: Step 33 Simplified logic, removed CI/Cert flows * Fixed: PIXIT.CGEN.FailsafeExpiryLengthSeconds hard coded to 1. * Updated: Step 22 to handle expected behavior. * Fixed: code-lints, removed unused import chip.exceptions.ChipStackError. * Fixed: Use ChipStackException and remove unnecessary assert.
This file contains hidden or 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
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.
Bumps third_party/pigweed/repo from
2e14574to8da9f28.Commits
8da9f28roll: host_toolsd7346deroll: gn66268faroll: qemu53a0fb1roll: ninja1ad00fedocs: Add favicon to pigweed.dev97a1fc5pw_presubmit: Exclude .ico from copyright check34d3f73pw_metric: Fix path to MetricService2b9ac8fpw_bloat: Fix double printing ASCII table4a117c7pw_console: Json Log file formatter9a7e6a4pw_bloat: Fix broken size report on pw_bloat docsDependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot mergewill merge this PR after your CI passes on it@dependabot squash and mergewill squash and merge this PR after your CI passes on it@dependabot cancel mergewill cancel a previously requested merge and block automerging@dependabot reopenwill reopen this PR if it is closed@dependabot closewill close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)