legit

Fork of https://git.icyphox.sh/legit

Commits at 8c0253211db0783714ac5f70340ba3b55d9e66b1

  1. d224558e Move embedded resources into dedicated module For testing. Child packages cannot access embedded resources. Shota FUJI authored at Shota FUJI comitted at
  2. eac513f5 Embed default templates into the binary https://tangled.org/pocka.jp/legit/issues/3 https://github.com/icyphox/legit/issues/10 Deployment & distribution convenience. Shota FUJI authored at Shota FUJI comitted at
  3. 0e496be9 Embed static files into the binary https://tangled.org/pocka.jp/legit/issues/3 Having to copy the "static/" directories is PITA, and distribution burden. Shota FUJI authored at Shota FUJI comitted at
  4. 208876a9 Don't generate bluemonday policy for each requests https://tangled.org/pocka.jp/legit/issues/4 bluemonday's documentation says the policy creation/editing is not safe to use in multiple goroutines. Now only one policy exists in the entire application and no creation/editing is happening inside HTTP server's goroutine. This also brings performance improvements. On my machine, spamming repo summary page took 10ms~11ms med (avg), but with this patch they are now 7ms~8ms med (avg). k6 run --vus 30 --iterations 1000 ./tests/k6/spam_repo_summary.js Thank Anthony Wang for sending the original patch that brought this topic in. Shota FUJI authored at Shota FUJI comitted at
  5. e39d5a3b Add link to issues page For transparency? Shota FUJI authored at Shota FUJI comitted at
  6. c2244176 Exclude unnecessary files from Nix source To prevent unnecessary rebuilds. Changes to README should not issue a new build. Shota FUJI authored at Shota FUJI comitted at
  7. afb93b77 Set "meta.mainProgram" in Nix derivation Nix outputs warning message when **user** calls "lib.getExe" on this package. --- evaluation warning: getExe: Package legit does not have the meta.mainProgram attribute. We'll assume that the main program has the same name for now, but this behavior is deprecated, because it leads to surprising errors when the assumption does not hold. If the package has a main program, please set `meta.mainProgram` in its definition to make this warning go away. Otherwise, if the package does not have a main program, or if you don't control its definition, use getExe' to specify the name to the program, such as lib.getExe' foo "bar". Shota FUJI authored at Shota FUJI comitted at
  8. 912daf3f Setup load testing It's minimal, but useful for future additions. Considering the current trend of malcious crawlers, I believe this is necessary. Shota FUJI authored at Shota FUJI comitted at
  9. 3ce7ca00 Eliminate boilerplate code from Flake file Shota FUJI authored at Shota FUJI comitted at
  10. 921f987b Migrate "defaultPackage" to "packages.default" (Nix Flakes) "defaultPackage" is deprecated. Shota FUJI authored at Shota FUJI comitted at
  11. 9d070622 Update vendorHash of Nix derivation Shota FUJI authored at Shota FUJI comitted at
  12. 932dc3e7 Add -server.host and -server.port CLI flags This is handy for development inside a container / VM. Shota FUJI authored at Shota FUJI comitted at
  13. fbc46772 Add footer The motivation is to display "Powered by ...". The patch I received adds that text to header, and I felt it stands out too much considering frequency of user using (reading and getting actually useful information from) it. I like the idea of "Powered by" as there were times that text helped me discover new tools. The most natural place to put it is footer, but my legit fork does not have that. And just having "Powered by ..." text alone feels off, so I added admin configurable links section for things like contact email address, mirrors, and other social links. I personally like the outcome. Shota FUJI authored at Shota FUJI comitted at
  14. 784362b7 Display message if no README found The info log is completely useless, as it's repo owner's fault for not having README, instead of site admin's. I configured this in the template so users (site admins) can customize their own view. Shota FUJI authored at Shota FUJI comitted at
  15. b801c06d Remove redundant return statements gopls yells at me. Shota FUJI authored at Shota FUJI comitted at
  16. 565beb7b Update chroma to highlight more languages in Markdown Anthony Wang authored at Shota FUJI comitted at
  17. 2b352672 Fix weird URL search parameter handling (raw=true) A query component in URL does not have to be "key=value". In fact, browsers interpret that as a URLSearchParams, a list of "key" or "key=value" allowing a same key appearing multiple times. You don't have to set "true" or "false": simply set the key or remove the one. Shota FUJI authored at Shota FUJI comitted at
  18. 09c5f871 Display checked-in image in Markdown preview Co-authored-by: Anthony Wang <a@unnamed.website> Shota FUJI authored at Shota FUJI comitted at
  19. 74ed6804 Format .nix files with nixfmt Shota FUJI authored at Shota FUJI comitted at
  20. d3443fb7 Add "gopls" to "nix develop" packages To stop requiring devs to install gopls globally. Shota FUJI authored at Shota FUJI comitted at
  21. 66fa2bb5 Add link for raw version of file The original patch also contains support for image display in READMEs. I (committer) removed the latter for brevity and revert-ability. Anthony Wang authored at Shota FUJI comitted at
  22. f72154de Fix binary file not available even with "raw" URL parameter This is critical for displaying images in README files. Shota FUJI authored at Shota FUJI comitted at
  23. 07103b7d Setup "nix fmt" There was no way to call dprint (except "nix-shell" and global mutation) Shota FUJI authored at Shota FUJI comitted at
  24. fab9af91 Upgrade nixpkgs The dprint CLI was too old a JSON dprint plugin denied to run. Shota FUJI authored at Shota FUJI comitted at
  25. 9eb33dfd Fix relative links in README markdown fails to resolve Shota FUJI authored at Shota FUJI comitted at
  26. 4efb241d Fix blob page code block slightly overflows vertically That makes unpleasant UX, such as almost-full scrollbars and requiring two-fingers swipe twice (one for scroll `<pre>` and second for the page) Shota FUJI authored at Shota FUJI comitted at
  27. 5e20b033 HTML preview for Markdown While Markdown is relatively readable format, syntax highlighted monospace font is not easy to read compared to well-spaced sans-serif. Shota FUJI authored at Shota FUJI comitted at
  28. d38d10b2 Increase top padding on 3-column layout I don't know why, but it was uncomfortable to read texts at the very near the top of a content area. Shota FUJI authored at Shota FUJI comitted at
  29. 291d38c0 Add more spaces to Markdown headings Those were so packed and hard to read. Shota FUJI authored at Shota FUJI comitted at
  30. 44c4ba1f Display datetime in user's time zone by using JavaScript Everytime I see datetime text with time zone I'm not familiar with, I struggle to calculate from my local time zone. There are situations where author's/committer's local time zone matters, (e.g. did they write this patch in night?) I kept the original datetime in "title" attribute (tooltip). Browsers' handling of "title" attribute is bad, but it's better than nothing. Shota FUJI authored at Shota FUJI comitted at