Copilot
Copilot
@samchon 👋 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...
> Add caching of permission priority. Use Caffeine cache. Cache for like 1 minute. You can use loading cache I think. Of course synchronous cache Added Caffeine cache with 1-minute...
> @copilot mock JavaPlugin#getName for tests Added mock for JavaPlugin#getName() in test setup. Commit: b34d017
@jeevbid I've opened a new pull request, #25545, to work on those changes. Once the pull request is ready, I'll request review from you.
> @copilot Based on the issue description, re-implement it. Re-implemented the solution with the following fixes in commit dbd2ebb: 1. Fixed `tspconfig.yaml` for RLC to include `is-modular-library: false` option (was...
> @copilot When you implement each step of the expression grammar refactor: > > - Update the grammar AND the corresponding visitors / planner code in the same change.... Understood....
> @copilot Let’s implement Step 1 using the non–left-recursive approach# Implemented Step 1 in commit 53c8c58. Grammar changes: - expr → orExpr → andExpr → notExpr → predicateExpr (explicit precedence...
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...
> Can you rebase and regenerate test data please? The test data is already up to date. I ran `make generate` and it produced no changes, which confirms that the...
> I have some more remarks @copilot please work on them. > > Also remove Python 3.9 from workflow in ci.yml Removed Python 3.9 from the CI workflow matrix and...