Copilot
Copilot
> @copilot you should create a separate destination called snap-conversions-v3 and not update the existing one. I've addressed your feedback and created a separate destination called `snap-conversions-v3` instead of updating...
@nicomt 👋 This repository doesn't have [Copilot instructions](https://docs.github.com/enterprise-cloud@latest/copilot/how-tos/configure-custom-instructions/add-repository-instructions?tool=webui). With Copilot instructions, I can understand the repository better, work faster and produce higher quality PRs. I can generate a .github/copilot-instructions.md file...
> @copilot - look at all CI failures - fix'em! Fixed! The issue was that Maven cannot resolve properties in the `` tag. I've updated the fix to: 1. Keep...
> Some additional cleanup is necessary Completed cleanup addressing all specific review feedback: removed fmt.Printf statements, optimized logger usage patterns, added constants for common fields, replaced errors.Wrap with fmt.Errorf, and...
> @copilot That's not the bug. If someone puts an unknown feature in the config file, it throws an exception unexpectedly The current implementation does exactly that - when an...
> @copilot It fails when an unknown feature is checked with `isFeatureEnabled` Fixed in 6bcd4db. The `isFeatureEnabled` method now warns and returns `null` for unknown features instead of throwing "Cannot...
> @copilot That was the fix we needed. Undo the earlier change. Done in 4514234. Reverted the `_validateFeatures` change - unknown features in config file now throw as before. Only...
> @copilot rebase and make it one commit I'm unable to perform interactive rebasing or force pushing in this environment. You'll need to squash the commits when merging the PR,...
> @copilot address my review comments I've addressed your review comments in commit 723105e. The app.babel.js file now uses ES6+ syntax throughout: ✅ **Template literals**: Converted the last remaining string...
> @copilot apply changes based on the comments in [this thread](https://github.com/OWASP-BLT/BLT/pull/5290#pullrequestreview-3575973064) All review comments have been addressed in commit 607692a: - Removed unused context variables from like_issue and dislike_issue -...