Skip to content
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.

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

doc: Add nRF54L15 to documentation #545

Merged
merged 4 commits into from
Apr 18, 2024
Merged

Conversation

totyz
Copy link
Contributor

@totyz totyz commented Apr 15, 2024

CI parameters

(branch, hash, pull/XXX/head)
NRF_revision=main

Description

doc: Remove memory_requirements.py script
doc: remove support for plantuml from build script
doc: Add nRF54L15

JIRA ticket: KRKNWK-18802

Self review

  • There is no commented code.
  • There are no TODO/FIXME comments without associated issue ticket.
  • Commits are properly organized.
  • Change has been tested.
  • Tests were updated (if applicable).

@totyz totyz requested review from ktaborowski and annwoj April 15, 2024 12:45
@github-actions github-actions bot added doc-required PR must not be merged without tech writer approval. scripts labels Apr 15, 2024
@totyz totyz removed the scripts label Apr 15, 2024
Copy link

github-actions bot commented Apr 15, 2024

Sample diff used total

Memory usage did not change for any of the samples.

.config diff

@totyz
Copy link
Contributor Author

totyz commented Apr 15, 2024

doc/samples/samples_list.rst Outdated Show resolved Hide resolved
doc/ncs_links_update.sh Outdated Show resolved Hide resolved
doc/samples/sid_end_device.rst Outdated Show resolved Hide resolved
doc/setting_up_sidewalk_environment/requirements.rst Outdated Show resolved Hide resolved
doc/setting_up_sidewalk_environment/requirements.rst Outdated Show resolved Hide resolved
doc/setting_up_sidewalk_environment/requirements.rst Outdated Show resolved Hide resolved
Copy link
Contributor

@ktaborowski ktaborowski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good

@totyz totyz force-pushed the pr_nrf54l_doc branch 2 times, most recently from ad97d42 to 36f0e86 Compare April 16, 2024 07:46
@totyz totyz requested a review from annwoj April 16, 2024 07:59
doc/samples/samples_list.rst Outdated Show resolved Hide resolved
doc/samples/samples_list.rst Outdated Show resolved Hide resolved
@totyz totyz requested a review from annwoj April 16, 2024 12:41
totyz added 3 commits April 17, 2024 12:44
Add nRF54L15 PDK to supported platforms.
Update memory requirements.
Update provisioning section about new address offset for nRF54L15 PDK

Co-authored-by: Anna Wojdyło <[email protected]>

Signed-off-by: Tomasz Tyzenhauz <[email protected]>
plantuml is not used anymore in documentation

Signed-off-by: Tomasz Tyzenhauz <[email protected]>
memroy requirements table is now generate in CI

Signed-off-by: Tomasz Tyzenhauz <[email protected]>
Move memory requirement to includes/memory_requirement.txt

Signed-off-by: Tomasz Tyzenhauz <[email protected]>
@totyz totyz merged commit b82a186 into nrfconnect:main Apr 18, 2024
56 checks passed
@totyz totyz deleted the pr_nrf54l_doc branch April 18, 2024 08:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc-required PR must not be merged without tech writer approval. scripts
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants