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

Exact match for boundary var attributes #3

Closed
wants to merge 10 commits into from

Conversation

Dave-Allured
Copy link
Owner

@Dave-Allured Dave-Allured commented Dec 2, 2020

CF section 7.1: On boundary variables, require exact match for CF defined attributes which affect the interpretation of values. Also clarify requirement for data types of such attributes.

Closes issue cf-convention#265

Release checklist

  • Authors updated in cf-conventions.adoc?
  • Next version in cf-conventions.adoc up to date? Versioning inspired by SemVer.
  • history.adoc up to date?
  • Conformance document up-to-date?

For maintainers

After the merge remember to delete the source branch.
Tags are set at the conclusion of the annual meeting; until then master always is a draft for the next version.

Sec. 7.1:  Require exact string match for functional attributes on boundary variables.
Issue cf-convention#265
Amend proposal, require exact value match, but data type may vary.
@Dave-Allured
Copy link
Owner Author

Amended just now. My original proposal required exact match by both value and data type. I relaxed the data type requirement.

Add two words, "such as", to generalize this list and include unpredicted cases.
Dave-Allured pushed a commit that referenced this pull request Oct 18, 2023
* more text following 2020-11-27 discussions

* bounds

* tidy

* tidy

* tidy

* tidy

* reproducability

* offset

* indices

* indices

* indices

* super
@Dave-Allured
Copy link
Owner Author

Closing without merge. Refer to PR cf-convention#467 for continuation.

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.

1 participant