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

Add support for variables of type string #138

Closed
wants to merge 3 commits into from

Conversation

JimBiardCics
Copy link
Contributor

@JimBiardCics JimBiardCics commented Jul 19, 2018

Change the Data Types section to include the string type and change the description of variables containing text data to include atomic string elements in addition to character arrays.

See http://cf-trac.llnl.gov/trac/ticket/XXX (Trac unavailable right now.)

  • Added link from trac ticket to this PR? (Trac unavailable right now.)

Applied via ​#138

  • [x ] Updated "Revision History"? (Use the date you applied the changes.)

Add support for the data type string to the list of support data types, and change the section talking about string variables to describe both character array and string variations.
Copy link
Contributor

@Dave-Allured Dave-Allured left a comment

Choose a reason for hiding this comment

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

(Comment deleted by author. See issue #139.)

@JimBiardCics
Copy link
Contributor Author

@Dave-Allured, I modified the wording. See what you think.

@ChrisBarker-NOAA
Copy link
Contributor

@JimBiardCics: I think the new working is great -- +1

Thanks!

@Dave-Allured
Copy link
Contributor

I agree with the new wording. Thanks, Jim.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants