-
Notifications
You must be signed in to change notification settings - Fork 0
Comparing changes
Open a pull request
base repository: levibostian/decaf
base: 0.8.0
head repository: levibostian/decaf
compare: 0.9.0
- 13 commits
- 19 files changed
- 2 contributors
Commits on Dec 17, 2025
-
refactor: replace jsr:deno-library/template with npm:ventojs for stri…
…ng template engine deno-library/template doesn't offer support for loops and I'll be needing that allowing users to provide a custom PR comment. I chose vento because... - very similar syntax to deno-library/template (using {{...}}) to be simple and easy to use. - gives more powerful features like loops, conditionals. - zero dependencies. small. simple.Configuration menu - View commit details
-
Copy full SHA for 3d08b6e - Browse repository at this point
Copy the full SHA 3d08b6eView commit details -
add inputs for providing a pull request comment template string. eith…
…er file or string.
Configuration menu - View commit details
-
Copy full SHA for c778541 - Browse repository at this point
Copy the full SHA c778541View commit details -
Configuration menu - View commit details
-
Copy full SHA for b9e4cc5 - Browse repository at this point
Copy the full SHA b9e4cc5View commit details -
refactor: document usage of ventojs for later reference
to help devs + agents learn ventojs quickly, putting documentation in the code via comments with basic usage.
Configuration menu - View commit details
-
Copy full SHA for 8f8f897 - Browse repository at this point
Copy the full SHA 8f8f897View commit details -
completely replace contents of PR comment each time, dont append
Because we're using string templates now, we don't want to use the appending feature of appending context to the existing PR comment. We want to completely replace it instead, just because then you don't have to add a bunch of logic to your string template.
Configuration menu - View commit details
-
Copy full SHA for ac88dce - Browse repository at this point
Copy the full SHA ac88dceView commit details -
Configuration menu - View commit details
-
Copy full SHA for f1a7105 - Browse repository at this point
Copy the full SHA f1a7105View commit details -
Configuration menu - View commit details
-
Copy full SHA for 814e6b4 - Browse repository at this point
Copy the full SHA 814e6b4View commit details -
fix: escape the pull request template string
When you pass in the template string as a param, there's a good chance that it uses different kinds of quotes inside of it, which then messes with the CLI Bash command that we're running, where we're trying to pass the string into decaf. So, as a way to properly escape that string you pass in, we write to a temporary file and then just pass that file to decaf.
Configuration menu - View commit details
-
Copy full SHA for 2123eb5 - Browse repository at this point
Copy the full SHA 2123eb5View commit details -
Configuration menu - View commit details
-
Copy full SHA for f0cb47d - Browse repository at this point
Copy the full SHA f0cb47dView commit details -
improve error message when ventojs is given an invalid template
The original error message I got when running the code in the CI was talking about regex, which is super confusing when I wasn't passing regex at all. So, I just think we need to have a better error message in general to help if users encounter a problem.
Configuration menu - View commit details
-
Copy full SHA for f43d182 - Browse repository at this point
Copy the full SHA f43d182View commit details -
refactor: all uses of ventojs in the codebase re-use the util.renderS…
…tringTemplate function
Configuration menu - View commit details
-
Copy full SHA for ec7c774 - Browse repository at this point
Copy the full SHA ec7c774View commit details -
Configuration menu - View commit details
-
Copy full SHA for c3ec1b3 - Browse repository at this point
Copy the full SHA c3ec1b3View commit details -
Configuration menu - View commit details
-
Copy full SHA for 0621668 - Browse repository at this point
Copy the full SHA 0621668View commit details
This comparison is taking too long to generate.
Unfortunately it looks like we can’t render this comparison for you right now. It might be too big, or there might be something weird with your repository.
You can try running this command locally to see the comparison on your machine:
git diff 0.8.0...0.9.0