Skip to content

Fix/resolve pr 870 conflict#871

Open
shanmukhkalasamudram wants to merge 9 commits into
developfrom
fix/resolve-pr-870-conflict
Open

Fix/resolve pr 870 conflict#871
shanmukhkalasamudram wants to merge 9 commits into
developfrom
fix/resolve-pr-870-conflict

Conversation

@shanmukhkalasamudram
Copy link
Copy Markdown
Contributor

Fixing merge conflicts

Mark Jensen and others added 9 commits November 3, 2024 22:18
- Merged develop into master.
- Cleaned up a large number of definitions, comments, inverse definitions, domains, and ranges across ontology modules.
- Revised temporal and event modeling, including interval relations, temporal ordering, process parthood, and several event definitions.
- Refactored information entity content, including updates to Information Bearing Artifact / Entity, document-related classes, and the introduction of Media Content Entity.
- Added and revised artifact, agent, measurement, transportation, infrastructure, and sensor-related content.
- Removed, deprecated, or reorganized outdated content, including familial relations and selected legacy labels/classes.
- Improved module/merge alignment and fixed ontology syntax and build issues.
- Added and repaired SPARQL QC queries for the release pipeline, including checks for labels, definitions, annotations, IRIs, and duplicate declarations.
- Added release/build tooling, including automated release notes generation and new make targets for diffing, version stamping, and merged builds.
- Expanded contributor documentation, style guidance, IRI registry support, and repository/site configuration.
This reverts commit 5d506a5.

:wq#
@johnbeve
Copy link
Copy Markdown
Contributor

There were concerns that there are differences in content, e.g. classes in merged on develop that aren't in the release and vice versa. I'm only seeing time updates in the commits at the moment. Are those the only differences you found?

@shanmukhkalasamudram
Copy link
Copy Markdown
Contributor Author

@johnbeve : There are content differences beyond timestamps (~130 IRIs in master not in develop, 5 in develop not in master). That's a separate issue from what this PR is fixing. This PR just resolves the merge conflict blocking #870. The content reconciliation of the merged file should be handled as a follow-up.

#870

Also, I can see a lot of changes accumulated in develop after the CCO PURLs PR was merged. Should we first coordinate with Mark to fix the CCO IRIs not resolving issue before merging #870 into master, or is it okay to proceed as-is?

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