Copilot
Copilot
> @copilot: Change in the spirit of #2466. Completed in 77ec3fc. Changed from direct attribute assignment to using `set_graph_attr()`, following the pattern from #2466 and the updated AGENTS.md guidelines. All...
> @copilot: Prefer `GATTR`, use explicit arguments instead of passing on ellipsis. Add ellipsis before optional arguments. Completed in 7f33188. Updated function signatures to use explicit arguments with ellipsis before...
> @copilot: See review, use `@inheritParams`, specify `GATTR` in `functions-R`.yaml . Completed in a5e6b2f. Added `check_dots_empty()` to `hex_lattice()`, specified GATTR in `functions-R.yaml`, and changed attribute name from `dims` to `dimvector`...
Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details. > [!WARNING] > > > Firewall rules blocked me...
> @copilot update only `packages/plugin-rsc/src` Updated to only apply hook filters to `packages/plugin-rsc/src`. Changes reverted for plugin-react and plugin-react-swc. Added filters to transform, resolveId, and load hooks in plugin-rsc. Commit:...
Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details. > [!WARNING] > > > Firewall rules blocked me...
> I don't think this is the issue. > Please look at this file https://github.com/yonahd/kor/blob/main/pkg/utils/slack.go I've examined the `pkg/utils/slack.go` file as you suggested. The `SendToSlack` method handles both webhook URLs...
> Take a look at #429 > I think the issue is the same Thank you for the clarification! After examining PR #429, I see that the fix was to...
> @copilot : there are still trimming errors: > > Run build\test-aot.ps1 'net9.0' ... Fixed all remaining trimming warnings. The issues were in DownstreamWebApiExtensions, MicrosoftIdentityAuthenticationMessageHandlerHttpClientBuilderExtensions, DownstreamApiExtensions, and Graph extensions that...
> @copilot. I left 2 comments. > There is also one meta-point (applying everywhere in this doc). "PreserveAuthority" is a MergedOptions property. It's not accessible to developers, nor settable through...