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 more 7.0 sample files #151

Merged
merged 12 commits into from
Mar 7, 2024
Merged

Add more 7.0 sample files #151

merged 12 commits into from
Mar 7, 2024

Conversation

dthaler
Copy link
Collaborator

@dthaler dthaler commented Mar 7, 2024

This PR is an attempt to factor PR #136 into multiple smaller PRs.
This PR includes files with no currently outstanding comments.

dthaler2 and others added 11 commits March 7, 2024 08:25
Copy 8 files from https://github.com/gedcom7code/test-files/tree/main/7
that exercise parts of GEDCOM 7.0 that aren't in other sample files.

Combined atsign.ged into the esiting escapes.ged.

Some files not copied include:
* char*.ged only make sense once there are corresponding 5.5.1 samples in the repository, since all 7.0 files
  are UTF-8 with a BOM.
* date-dual.ged likewise only makes sense as a 7.0 derivative of a 5.5.1 sample.
* enum-ext.ged likewise looked like it only made sense along with a 5.5.1 sample.
* obsolete.ged and tiny.ged are the same as the existing minimal70.ged.

Addresses part of issue FamilySearch#132

Signed-off-by: Dave Thaler <[email protected]>
Filed
FamilySearch/GEDCOM#420
while working on this.

Signed-off-by: Dave Thaler <[email protected]>
Signed-off-by: Dave Thaler <[email protected]>
Signed-off-by: Dave Thaler <[email protected]>
Signed-off-by: Dave Thaler <[email protected]>
Signed-off-by: Dave Thaler <[email protected]>
Signed-off-by: Dave Thaler <[email protected]>
dthaler pushed a commit to dthaler/GEDCOM that referenced this pull request Mar 7, 2024
The proposed recommendation is consistent with sample files:
* https://gedcom.io/testfiles/gedcom70/maximal70.ged has MEDIA, OTHER,
  and ELECTRONIC.
* FamilySearch/GEDCOM.io#151 adds obje-1.ged
  with PHOTO and VIDEO, as copied from
  https://github.com/gedcom7code/test-files/blob/main/7/obje-1.ged

Fixes FamilySearch#316 since it addresses the last piece that wasn't already done

Signed-off-by: Dave Thaler <[email protected]>
dthaler added a commit to FamilySearch/GEDCOM that referenced this pull request Mar 7, 2024
The proposed recommendation is consistent with sample files:
* https://gedcom.io/testfiles/gedcom70/maximal70.ged has MEDIA, OTHER,
  and ELECTRONIC.
* FamilySearch/GEDCOM.io#151 adds obje-1.ged
  with PHOTO and VIDEO, as copied from
  https://github.com/gedcom7code/test-files/blob/main/7/obje-1.ged

Fixes #316 since it addresses the last piece that wasn't already done

Signed-off-by: Dave Thaler <[email protected]>
Co-authored-by: Dave Thaler <[email protected]>
_pages/tools.md Outdated Show resolved Hide resolved
@dthaler
Copy link
Collaborator Author

dthaler commented Mar 7, 2024

Ok to merge after renaming lang-all.ged to lang.ged, per discussion March 7, 2024

@dthaler dthaler merged commit 3d13a2e into FamilySearch:main Mar 7, 2024
@dthaler dthaler deleted the test-files-1 branch March 7, 2024 21:05
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