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

Update generated files #165

Merged
merged 2 commits into from
May 26, 2024
Merged

Update generated files #165

merged 2 commits into from
May 26, 2024

Conversation

github-actions[bot]
Copy link
Contributor

Update generated files
This PR is auto-generated by [gh pr create].

Copy link
Collaborator

@dthaler dthaler left a comment

Choose a reason for hiding this comment

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

Changes are still going backwards

@github-actions github-actions bot force-pushed the generate-main-files branch from 54c01e7 to 689b411 Compare May 4, 2024 21:04
@github-actions github-actions bot force-pushed the generate-main-files branch 2 times, most recently from 6eefe72 to df1f73f Compare May 18, 2024 21:04
@github-actions github-actions bot force-pushed the generate-main-files branch from df1f73f to 0f5f76b Compare May 25, 2024 21:04
@dthaler
Copy link
Collaborator

dthaler commented May 26, 2024

After investigating, this PR is doing the right thing.
The main branch of GEDCOM.io is intended to match the latest GEDCOM release (per discussion on PR #119) not the latest GEDCOM main branch.
The changes going backwards are ones added to GEDCOM (in PRs 436, 440, 454) since the 7.0.14 release.

@dthaler dthaler dismissed their stale review May 26, 2024 16:56

Resolved

This will re-trigger the validate-yaml workflow to run so that the PR
can be merged.

Signed-off-by: Dave Thaler <[email protected]>
@dthaler dthaler merged commit c09a02e into main May 26, 2024
1 check passed
@dthaler dthaler deleted the generate-main-files branch May 26, 2024 17:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants