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

Tidy installation SQL script(s) for beta #4020

Open
4 tasks
mathemancer opened this issue Nov 7, 2024 · 0 comments
Open
4 tasks

Tidy installation SQL script(s) for beta #4020

mathemancer opened this issue Nov 7, 2024 · 0 comments
Assignees
Labels
ready Ready for implementation restricted: maintainers Only maintainers can resolve this issue type: maintenance Refactoring and technical debt payoff work: db-layer Related to SQL or PL/pgSQL

Comments

@mathemancer
Copy link
Contributor

Problem

We made a lot of changes to the SQL scripts for beta, and they're currently a bit of a mess.

Proposed solution

We should:

  • Fix up the "reset" function to be safer and more comprehensive
  • Audit the current state of the __msar namespace and remove it if possible
  • Reorganize and tidy the functions in msar to be more maintainable
  • Consider moving all SQL to a single script for simplicity and ease of testing
@mathemancer mathemancer added type: maintenance Refactoring and technical debt payoff ready Ready for implementation restricted: maintainers Only maintainers can resolve this issue work: db-layer Related to SQL or PL/pgSQL labels Nov 7, 2024
@mathemancer mathemancer added this to the v0.2.0 (beta release) milestone Nov 7, 2024
@mathemancer mathemancer self-assigned this Nov 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready Ready for implementation restricted: maintainers Only maintainers can resolve this issue type: maintenance Refactoring and technical debt payoff work: db-layer Related to SQL or PL/pgSQL
Projects
None yet
Development

No branches or pull requests

1 participant