Skip to content

Conversation

@vrslev
Copy link
Collaborator

@vrslev vrslev commented Jul 5, 2025

Closes #96

@vrslev
Copy link
Collaborator Author

vrslev commented Jul 6, 2025

I’ve used Gemini CLI to code the core of this. Sadly, the free version after couple of minutes switched from pro to flash model, and started giving bad results. I’ll continue later with another agentic coding tool

@vrslev vrslev marked this pull request as ready for review July 7, 2025 12:56
@vrslev
Copy link
Collaborator Author

vrslev commented Jul 7, 2025

Claude and Gemini didn't help much. Same for Qwen3 32B with continue or cline

@vrslev
Copy link
Collaborator Author

vrslev commented Jul 7, 2025

They helped to add initial implementation, but it took so much time... Perhaps, the task is too complex for state-of-the-art models

@vrslev vrslev merged commit 8900cc3 into main Jul 7, 2025
8 checks passed
@vrslev vrslev deleted the ignore-global-vars branch July 7, 2025 13:02
@vrslev vrslev changed the title Ignore global vars (#96) Add support for global variables (#96) Jul 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add Final for global constants based on heuristics

2 participants