Skip to content
This repository has been archived by the owner on Sep 23, 2023. It is now read-only.

Bump Evidence to latest #23

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Conversation

archiewood
Copy link

No description provided.

@archiewood
Copy link
Author

As a side point, @gwenwindflower perhaps we should add a deployment for this project, eg on Netlify or Vercel?

It would:

  • Allow people to see what the reports would look like before booting up the codespace
  • Test that any changes to the evidence reporting still work as expected before merging PRs

@gwenwindflower
Copy link
Contributor

@archiewood yea i think a vercel deployment would be great, i can add that!

@archiewood
Copy link
Author

archiewood commented May 2, 2023

Now I think about it, it may have to be Netlify, there's currently a bug with Vercel x DuckDB:

https://docs.evidence.dev/deployment/vercel

@gwenwindflower
Copy link
Contributor

oh darn -- we just moved to Vercel from Netlify -- I think we have both for now, I believe I could put it on Netlify for the time being and longterm it can be on Vercel if that bug gets sorted.

@gwenwindflower
Copy link
Contributor

@archiewood okay looks like we do indeed have Netlify still for now, i can work on a deploy next week

@gwenwindflower
Copy link
Contributor

closes #13

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants