Skip to content

Greatly simplify the NX_CHAR documentation#1482

Merged
prjemian merged 1 commit intomainfrom
simple_char
Sep 29, 2024
Merged

Greatly simplify the NX_CHAR documentation#1482
prjemian merged 1 commit intomainfrom
simple_char

Conversation

@phyy-nx
Copy link
Copy Markdown
Contributor

@phyy-nx phyy-nx commented Sep 29, 2024

Idea is to state that NX_CHAR is a string of characters, and remove the other documentation that doesn't seem as necessary nowadays.

Fixes #1441

Idea is to state that NX_CHAR is a string of characters, and remove the other documentation that doesn't seem as necessary nowadays.

Fixes #1441
@phyy-nx phyy-nx requested a review from prjemian September 29, 2024 08:41
Copy link
Copy Markdown
Contributor

@prjemian prjemian left a comment

Choose a reason for hiding this comment

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

Great. I, too, believe the additional text is no longer necessary.

@prjemian prjemian added the NIAC has requested The NIAC has requested this issue to be considered label Sep 29, 2024
@prjemian prjemian added this to the NXDL 2025 milestone Sep 29, 2024
@prjemian prjemian assigned prjemian and phyy-nx and unassigned prjemian Sep 29, 2024
@prjemian prjemian merged commit ab5e65d into main Sep 29, 2024
@prjemian prjemian deleted the simple_char branch September 29, 2024 13:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

NIAC has requested The NIAC has requested this issue to be considered

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

scalar or list values for Fields/Attributes, but does it work for NX_CHAR, too?

2 participants