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

General Overhaul #32

Merged
merged 11 commits into from
Aug 23, 2023
Merged

General Overhaul #32

merged 11 commits into from
Aug 23, 2023

Conversation

cmheazel
Copy link
Collaborator

@cmheazel cmheazel commented Aug 21, 2023

Aligned with updates to Master. Cleaned up citations and definitions. General editorial cleanup. Removed UML model for use in a separate document (tentative 23-050). Should be ready for DWG review.

Note: The abstract-specification directory was retained to resolve conflicts on the pull request. It will be deleted once this pull request is merged.

Re-structured definitions and references making liberal use of annexes.
Updated UML model, added diagrams, added software generated data dictionaries.
This version should align with the current version of the Master branch.
@cmheazel cmheazel closed this Aug 22, 2023
@cmheazel cmheazel reopened this Aug 23, 2023
cmheazel and others added 5 commits August 23, 2023 07:46
Copied 06-other-clauses.adoc from Master.
Moved 06-other-clauses to its original location in the directory tree.
metanorma.yml did not reference the new path and file name (23-049/23-049.adoc)
Copy link
Contributor

@chris-little chris-little left a comment

Choose a reason for hiding this comment

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

Looks safe. One spelling typo by me.

@chris-little
Copy link
Contributor

as agreed at Temporal DWG 2023-08-22, so that there is one branch for wider review by @samadammeek and @ronaldtse and others.

@chris-little chris-little merged commit 790a938 into master Aug 23, 2023
2 checks passed
@chris-little chris-little deleted the August-2023 branch August 23, 2023 12:14
Copy link
Contributor

@chris-little chris-little left a comment

Choose a reason for hiding this comment

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

Looks OK.

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.

2 participants