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: OS support: update table #17214

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

greg-fer
Copy link
Contributor

@greg-fer greg-fer commented Sep 6, 2024

Updated table with the finalized input received from stakeholders. NCSDK-29043.


  • Remove TBDs from tables. Info from @shanthanordic : "more information is expected in Sep. macOS15 release is in Oct time frame."

@greg-fer greg-fer added doc-required PR must not be merged without tech writer approval. CI-disable Disable CI for this PR doc only labels Sep 6, 2024
@greg-fer greg-fer added this to the 2.8.0 milestone Sep 6, 2024
@github-actions github-actions bot added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Sep 6, 2024
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Sep 6, 2024

CI Information

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

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

@greg-fer greg-fer force-pushed the doc_update_supported_platforms branch from 76f383b to 48bf4c0 Compare September 6, 2024 07:02
@github-actions github-actions bot removed the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Sep 6, 2024
@greg-fer greg-fer force-pushed the doc_update_supported_platforms branch 2 times, most recently from a20986f to 0bffd4b Compare September 6, 2024 09:58
@greg-fer greg-fer marked this pull request as draft September 6, 2024 09:59
@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.

@greg-fer greg-fer force-pushed the doc_update_supported_platforms branch 2 times, most recently from 6f21b24 to 19a0d01 Compare September 20, 2024 11:07
@greg-fer greg-fer force-pushed the doc_update_supported_platforms branch 3 times, most recently from e0561be to e8fd38e Compare September 20, 2024 15:19
@greg-fer greg-fer marked this pull request as ready for review September 23, 2024 14:49
@greg-fer greg-fer requested review from a team as code owners September 23, 2024 14:49
doc/nrf/links.txt Outdated Show resolved Hide resolved
@greg-fer greg-fer force-pushed the doc_update_supported_platforms branch from a944eb0 to c77c1d8 Compare September 25, 2024 12:14
@greg-fer greg-fer force-pushed the doc_update_supported_platforms branch from c77c1d8 to 3e660cf Compare September 25, 2024 13:04
doc/nrf/installation/recommended_versions.rst Outdated Show resolved Hide resolved
doc/nrf/installation/recommended_versions.rst Outdated Show resolved Hide resolved
doc/nrf/installation/recommended_versions.rst Outdated Show resolved Hide resolved
doc/nrf/installation/recommended_versions.rst Outdated Show resolved Hide resolved
doc/nrf/installation/recommended_versions.rst Outdated Show resolved Hide resolved
doc/nrf/installation/recommended_versions.rst Outdated Show resolved Hide resolved
doc/nrf/installation/recommended_versions.rst Outdated Show resolved Hide resolved
* Updated the required `SEGGER J-Link`_ version to v7.94i.
* Updated:

* The required `SEGGER J-Link`_ version to v7.94i.
Copy link
Contributor

Choose a reason for hiding this comment

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

Would you like to convert the v7.94i to a shortcut in shortcut.txt? That way it could be also used by other documents.

Copy link
Contributor Author

@greg-fer greg-fer Sep 30, 2024

Choose a reason for hiding this comment

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

Not in changelog, because the shortcut introduces backward compatibility: this SDK version supports 7.94i, the next SDK version will support a different J-Link version, which will not be backward compatible, but with the shortcut, the version will change also in old release notes. (Also, this PR is not about J-Link version, I'm just changing formatting.)

@greg-fer greg-fer force-pushed the doc_update_supported_platforms branch from 3e660cf to f9d1e0d Compare September 30, 2024 13:49
@greg-fer greg-fer marked this pull request as draft September 30, 2024 14:40
@greg-fer
Copy link
Contributor Author

Pending conversation results between myself, Kari, and Shantha.

@greg-fer greg-fer force-pushed the doc_update_supported_platforms branch from f6fa8ae to b1d10f0 Compare October 3, 2024 09:54
@greg-fer
Copy link
Contributor Author

greg-fer commented Oct 3, 2024

Pending conversation results between myself, Kari, and Shantha.

Summary of discussed changes:

  • Have only 1 table with OS support for FW and remove the tier definitions -- this is to keep it short and clear and within the context of the SDK
  • Move the OS support for tools to their own documentation and link to there (nRF Util) -- this is because tools means just nRF Util, VS Code extension, and nRF Connect for Desktop (which both use nRF Util), not the entirety of the toolchain

@greg-fer greg-fer force-pushed the doc_update_supported_platforms branch 4 times, most recently from 15ef49a to 17389a6 Compare October 3, 2024 13:59
@thst-nordic thst-nordic self-requested a review October 4, 2024 08:32
Copy link
Contributor

@thst-nordic thst-nordic left a comment

Choose a reason for hiding this comment

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

simplified table looks good.

doc/nrf/installation/recommended_versions.rst Outdated Show resolved Hide resolved
@greg-fer greg-fer marked this pull request as ready for review October 4, 2024 08:48
- Built and tested with :ref:`Twister <running_unit_tests>`.
- Not supported.
* - `Linux - Ubuntu 22.04 LTS`_
- Built and tested with :ref:`Twister <running_unit_tests>`. Comprehensive testing with Nordic Semicondutor hardware.
Copy link
Contributor

Choose a reason for hiding this comment

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

Semicondutor --> Semiconductor

Updated table with input received from stakeholders.
Updated definitions based on receved feedback.
NCSDK-29043.

Signed-off-by: Grzegorz Ferenc <[email protected]>
@greg-fer greg-fer force-pushed the doc_update_supported_platforms branch from 17389a6 to 7c03afe Compare October 4, 2024 09:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI-disable Disable CI for this PR doc only doc-required PR must not be merged without tech writer approval.
Projects
None yet
Development

Successfully merging this pull request may close these issues.