Conversation
|
Great job! No new security vulnerabilities introduced in this pull request |
|
Note I intentionally published with the intent of going back to draft state to test the |
theMickster
left a comment
theMickster
left a comment
There was a problem hiding this comment.
🛑 All changes need to bump the plugin's version. The change is rather small so a patch bump feels most appropriate. On your local workstation, in the ai-plugins folder, you will execute the following:
./scripts/bump-plugin-version.sh bitwarden-software-engineer <next-patch-version>🛑 All changes need an entry in plugins/bitwarden-software-engineer/CHANGELOG.md.
Suggested entry under ### Changed:
- Updated
writing-database-queriesskill: clarified dual-ORM architecture, rewrote EDD section to reflect no-rollback deployment model, documented stored procedure compatibility patterns, and simplified key locations.
…i/update-writing-database-queries-skill
Address PR #56 review feedback: add version bump and changelog entry for writing-database-queries skill update.
- Remove "For example" prefix from stored procedure section - Update critical rule to match dual-ORM exceptions documented above
theMickster
left a comment
theMickster
left a comment
There was a problem hiding this comment.
One small, but I think, important wording change and I think we are good to publish & likely merge.
withinfocus
left a comment
withinfocus
left a comment
There was a problem hiding this comment.
Just one oversight and then this is good.
| @@ -0,0 +1,5 @@ | |||
| { | |||
3 participants
Draft: updating
writing-database-queries skill content