Closed SmittieC closed 1 week ago
cc @bderenzi
@coderabbitai do a full review
@SmittieC: I will perform a full review of the pull request.
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Attention: Patch coverage is 96.96970%
with 1 line
in your changes missing coverage. Please review.
:white_check_mark: All tests successful. No failed tests found.
Files with missing lines | Patch % | Lines |
---|---|---|
apps/experiments/models.py | 80.00% | 1 Missing :warning: |
:loudspeaker: Thoughts on this report? Let us know!
π¨ Try these New Features:
- One "issue" with this is that if we create a new version, then chatting to a working version will still tag responses with the "unreleased" tag even though there are no changes between the current working version and the latest version.
I don't think this is worth trying to work around.
Generally seems OK though I wonder about using a separate tag for the 'unreleased' vs changing the name of the version tag e.g.
v3 unreleased
.
I like this suggestion. I'm going with it
Description
Options:
User Impact
Demo
Session table shows versions that the session spans
We indicate that the chat happened on an unreleased version
Docs
N/A