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

Remove path_package_database() #980

Closed
signekb opened this issue Jan 21, 2025 · 0 comments · Fixed by #985
Closed

Remove path_package_database() #980

signekb opened this issue Jan 21, 2025 · 0 comments · Fixed by #985
Assignees
@signekb

Comments

@signekb
Copy link
Member

signekb commented Jan 21, 2025

Since with the current setup, we wont create a db.
@signekb signekb self-assigned this Jan 21, 2025
This was referenced Jan 21, 2025
Merged
Merged
@signekb signekb moved this from Todo to In Review in Team project planning Jan 21, 2025
lwjohnst86 added a commit that referenced this issue Jan 22, 2025
@signekb @lwjohnst86
chore: 🔥 remove path_package_database() (#985)
bf14f30 
## Description

This PR removes `path_package_database()`and removes it from tests. It
also removes it from the docs.

Closes #980 

<!-- Select quick/in-depth as necessary -->
This PR needs a quick review.

## Checklist

- [X] Added or updated tests
- [X] Updated documentation
- [ ] Ran `just run-all`

---------

Co-authored-by: Luke W. Johnston <[email protected]>
@github-project-automation github-project-automation bot moved this from In Review to Done in Team project planning Jan 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@signekb signekb

Labels
None yet
Projects
Team project planning
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

chore: 🔥 remove path_package_database() seedcase-project/seedcase-sprout
1 participant
@signekb