Copilot
Copilot
@rom1504 👋 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 please make sure to start by adding a test reproducing the issue Started with test-first approach as requested. Added `useItemIssueRepro.js` that reproduces the issue (wrong packet format sent...
> @copilot revert all your tentative fix and your test then read the initial issue again and do a very minimal test reproducing whether activate item works. Do not check...
> @copilot see my latest comments Addressed both comments: improved test to check actual server-side effects instead of just local state, and added fallback for missing feature flag with version...
> @copilot if a user runs `azd login` or `azd logout` please return a message directing them to use the related `auth login/logout` commands. Done! When users run `azd login`...
> @copilot There's a lot of duplicated code between this new `AuditCommand` and `MakePotCommand`. Maybe the former could just extend or instantiate the latter without actually creating a POT file,...
> @copilot With the audit command, when a format such as JSON is used, there shouldn't be any additional output such as "Plugin file detected." or "Warning: Found 1 issue."...
> @copilot There's one failing test remaining: > > ``` ... Fixed the `--ignore-domain` flag handling to match the parent class behavior. When this flag is set, the domain is...
@JaydipGabani I've opened a new pull request, #650, to work on those changes. Once the pull request is ready, I'll request review from you.
> You know what? Maybe, in this case, there is no point to show an error with the ability to open a bug since it's a local error > Maybe...