Skip to content
This repository has been archived by the owner on Oct 4, 2024. It is now read-only.

Use sqlite, remove prover #98

Merged
merged 12 commits into from
Aug 11, 2023
Merged

Use sqlite, remove prover #98

merged 12 commits into from
Aug 11, 2023

Conversation

fkrause98
Copy link
Contributor

@fkrause98 fkrause98 commented Aug 11, 2023 •
edited
Loading

Changes:

  • Replace postgres with sqlite.
  • Remove prover submodule.
  • Go back to using esbuild to avoid node/npm dependency

How to test:

  • Cleanly clone the repo.
  • Checkout to this branch.
  • Run make setup run.
    The explorer should start without issues.

@fkrause98 fkrause98 changed the title Use sqlite remove prover Use sqlite, remove prover Aug 11, 2023
@klaus993 klaus993 temporarily deployed to testing August 11, 2023 18:34 — with GitHub Actions Inactive
@fkrause98 fkrause98 mentioned this pull request Aug 11, 2023
Open
juanbono
juanbono approved these changes Aug 11, 2023
View reviewed changes
Copy link
Contributor

@juanbono juanbono left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, please add the 3 required ENV variables to the README

@fkrause98 fkrause98 merged commit a7c76b5 into main Aug 11, 2023
1 check passed
@fkrause98 fkrause98 deleted the use-sqlite-remove-prover branch August 11, 2023 20:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@juanbono juanbono juanbono approved these changes

@klaus993 klaus993 Awaiting requested review from klaus993

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@fkrause98 @juanbono @klaus993