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

Closes #53 with simple tests (this obsoletes PR #57 and PR #65) #68

Merged
merged 6 commits into from
Jun 12, 2024

Conversation

fpl
Copy link
Contributor

@fpl fpl commented Nov 18, 2023

As asked, this is a new edition rebased on the current master.

shawnlaffan and others added 5 commits December 19, 2023 19:13
Also generate two features to ensure the
write works properly.  Otherwise a single
feature was being written but followed
by a failure.
Refactor the expected structure in the process.
@shawnlaffan
Copy link
Collaborator

@fpl - is this still an open PR?

#53 was closed by fpl@efd7313 but the t/transform.t file was not part of the associated PR.

@fpl
Copy link
Contributor Author

fpl commented Jun 12, 2024

It is still open: it implements with is_deeply using the t/transform.t test for #53. Maybe force-rebasing is not a great idea when master copies diverge, indeed....

@shawnlaffan shawnlaffan merged commit c782ae7 into ajolma:master Jun 12, 2024
6 checks passed
@shawnlaffan
Copy link
Collaborator

PR merged. Thank you.

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