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

Use data from URL for documentation #10

Open
roaldarbol opened this issue Sep 10, 2024 · 1 comment
Open

Use data from URL for documentation #10

roaldarbol opened this issue Sep 10, 2024 · 1 comment
Labels
documentation Improvements or additions to documentation
Milestone

Comments

@roaldarbol
Copy link
Owner

roaldarbol commented Sep 10, 2024

Currently the package is being massively inflated by the data that lives in the package. The alternative is to host the data elsewhere (https://github.com/roaldarbol/movement-data). However, there's a few hurdles:

  • Currently, it fails the reader validation checks
  • Exceptions need to be written in those validators
  • New validators need to be written for URLs

I have already substituted the data used in tests for minimal 20-line versions that live in tests/data.

@roaldarbol roaldarbol added the documentation Improvements or additions to documentation label Sep 10, 2024
@roaldarbol roaldarbol added this to the animovement v1.0 milestone Sep 15, 2024
@roaldarbol
Copy link
Owner Author

Actually just need to do this for documentation; for tests I use the header and like 10 lines of data.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
Status: 📝 Todo
Development

No branches or pull requests

1 participant