Skip to content

Conversation

@dthaler
Copy link
Contributor

@dthaler dthaler commented Oct 27, 2025

The same fix was already made in
GEDCOM.io/testfiles/gedcom70/filename-ged

PR #28 needs to be merged first, for the test to pass.

@tychonievich
Copy link
Contributor

@dthaler Even after merging #28 this still fails with the same dotnet error

@dthaler
Copy link
Contributor Author

dthaler commented Oct 28, 2025

@dthaler Even after merging #28 this still fails with the same dotnet error

I will rebase it, since the last run against this PR still shows:

Invoke-WebRequest https://github.com/ArmidaleSoftware/gedcom7/releases/download/v1.1.0/Windows-Release-GedValidate.zip -OutFile GedValidate.zip
which was pre-PR #28

The same fix was already made in
GEDCOM.io/testfiles/gedcom70/filename-ged

Signed-off-by: Dave Thaler <dthaler1968@gmail.com>
@dthaler
Copy link
Contributor Author

dthaler commented Oct 28, 2025

I rebased and the test now passes.

@tychonievich tychonievich merged commit c5d4a2c into gedcom7code:main Oct 29, 2025
1 check passed
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