Skip to content

Conversation

@gmarull
Copy link
Member

@gmarull gmarull commented Oct 2, 2024

No description provided.

Add warnings_filter extensions from Zephyr, now that it is gone from
upstream.

Signed-off-by: Gerard Marull-Paretas <[email protected]>
@gmarull gmarull requested review from a team as code owners October 2, 2024 12:09
@github-actions github-actions bot added doc-required PR must not be merged without tech writer approval. manifest changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels Oct 2, 2024
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Oct 2, 2024

The following west manifest projects have been modified in this Pull Request:

Name Old Revision New Revision Diff
nrfxlib nrfconnect/sdk-nrfxlib@b2ff80c nrfconnect/sdk-nrfxlib@380330c (main) nrfconnect/[email protected]

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@gmarull gmarull added the CI-disable Disable CI for this PR label Oct 2, 2024
@gmarull gmarull requested a review from rugeGerritsen October 2, 2024 12:10
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Oct 2, 2024

CI Information

To view the history of this post, clich the 'edited' button above
Build number: 2

Inputs:

Sources:

more details

Github labels

Enabled Name Description
ci-disabled Disable the ci execution
ci-all-test Run all of ci, no test spec filtering will be done
ci-force-downstream Force execution of downstream even if twister fails
ci-run-twister Force run twister
ci-run-zephyr-twister Force run zephyr twister
List of changed files detected by CI (0)

Outputs:

Toolchain

Version:
Build docker image:

Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped; ⚠️ Quarantine

  • ◻️ Toolchain
  • ◻️ Build twister
  • ◻️ Integration tests

Note: This message is automatically posted and updated by the CI

@NordicBuilder
Copy link
Contributor

You can find the documentation preview for this PR at this link. It will be updated about 10 minutes after the documentation build succeeds.

Note: This comment is automatically posted by the Documentation Publishing GitHub Action.

Add a new parameter, `warnings_filter_builders`, that allows to only
apply the filtering mechanism to certain builders. This can be useful
if certain warnings only show up in a particular builder, like our
custom `inventory` one.

Signed-off-by: Gerard Marull-Paretas <[email protected]>
This allows to filter out false warnings due to circular dependencies
with the nrf docset when building the inventory.

Signed-off-by: Gerard Marull-Paretas <[email protected]>
So that we can finally have clean nrfxlib builds.

Signed-off-by: Gerard Marull-Paretas <[email protected]>
We can finally build nrfxlib in a clean way.

Signed-off-by: Gerard Marull-Paretas <[email protected]>
@gmarull gmarull force-pushed the nrfxlib-final-fixes branch from 51d59a3 to 6c46a49 Compare October 2, 2024 12:28
@NordicBuilder NordicBuilder removed the DNM label Oct 2, 2024
@gmarull gmarull merged commit 29df534 into nrfconnect:main Oct 2, 2024
13 of 14 checks passed
@gmarull gmarull deleted the nrfxlib-final-fixes branch October 2, 2024 12:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. CI-disable Disable CI for this PR doc-required PR must not be merged without tech writer approval. manifest manifest-nrfxlib

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants