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

Spatial/Series refactor tracker #69

Closed
5 tasks done
Lun4m opened this issue May 6, 2024 · 1 comment · Fixed by #76
Closed
5 tasks done

Spatial/Series refactor tracker #69

Lun4m opened this issue May 6, 2024 · 1 comment · Fixed by #76
Labels
tech debt internal/maintainability improvements

Comments

@Lun4m
Copy link
Collaborator

Lun4m commented May 6, 2024

Per @intarga comment on #66.

@intarga intarga added the tech debt internal/maintainability improvements label Sep 25, 2024
@intarga
Copy link
Member

intarga commented Sep 25, 2024

Updates to this issue as things have changed.

Points 2 and 3 are resolved by #76

Point 4 is no longer valid, since we decided that filtering should happen inside the relevant tests themselves. This becomes an Olympian issue.

Point 5 I'm splitting out into a down-prioritised new issue (#77), since I realised validate_fresh is just a special case of validate (earlier referred to as validate_periodic). It's probably still worth doing, but I'm considering it as an optimisation rather than a structural blocker.

As a consequence I'm marking this whole issue as resolved by #76

@intarga intarga linked a pull request Sep 25, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tech debt internal/maintainability improvements
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants