Closed andylolz closed 2 months ago
I think this is ready to approve?
I think this is ready to approve?
All the things raised so far are hopefully addressed. If you’re happy then I’m happy!
I'm also happy for this to be merged! I haven't had time to do a line-by-line review, but the dev branch has been working locally, and I can't see anything here that would break it. Let us merge!
Amazing – thanks both 🎉
This merges
dev
intomain
, which includes lots of fixes:Fixes #20. Fixes #21. Fixes #23. Refs #42 (makes progress on this one, but possibly we should keep it open because this doesn’t exactly address it). Fixes #62. Fixes #66. Fixes #71. Fixes #77.
With #76 included in here, this also:
Fixes #61. Fixes #70.
How to deploy
I think that should do the trick?
Pull request checklist
main