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.

Sign up for GitHub

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

Jump to bottom

switch back to lazy loading of data #76

Closed
egouldo opened this issue Aug 14, 2024 · 0 comments · Fixed by #82
Closed

switch back to lazy loading of data #76

egouldo opened this issue Aug 14, 2024 · 0 comments · Fixed by #82
Assignees
@egouldo
Labels
breaking change ☠️ API change likely to affect existing code upkeep maintenance, infrastructure, and similar
Milestone
Respond Peer Revi...

Comments

@egouldo
Copy link
Owner

egouldo commented Aug 14, 2024

After implementing egouldo/ManyEcoEvo#110 revert back to lazily loading data with package namespace calls
@egouldo egouldo added breaking change ☠️ API change likely to affect existing code upkeep maintenance, infrastructure, and similar labels Aug 14, 2024
@egouldo egouldo added this to the Respond Peer Review Comments milestone Aug 14, 2024
@egouldo egouldo self-assigned this Aug 14, 2024
egouldo added a commit that referenced this issue Sep 1, 2024
@egouldo
bug!: fix #76 don't call data(*) after reverting to lazy loading da…
e7c8bce 
…ta in `ManyEcoEvo::`

---

refs: egouldo/ManyEcoEvo#110
@egouldo egouldo closed this as completed Sep 1, 2024
@egouldo egouldo mentioned this issue Sep 17, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@egouldo egouldo

Labels
breaking change ☠️ API change likely to affect existing code upkeep maintenance, infrastructure, and similar
Projects
None yet
Milestone
Respond Peer Review Comments
Development

Successfully merging a pull request may close this issue.

Rebuild manuscript egouldo/ManyAnalysts
1 participant
@egouldo