Skip to content

Remove redundant file open in build.py#14

Merged
alejandroklever merged 2 commits intofeature/implement-testsfrom
copilot/sub-pr-9-again
Mar 3, 2026
Merged

Remove redundant file open in build.py#14
alejandroklever merged 2 commits intofeature/implement-testsfrom
copilot/sub-pr-9-again

Conversation

Copy link

Copilot AI commented Feb 28, 2026

build.py was opening pyjapt/__init__.py but never reading it — the version was already available via pyjapt.__version__.

Change

  • Replaced the unused with open("pyjapt/__init__.py", "r") as f: block with a direct assignment:
# Before
with open("pyjapt/__init__.py", "r") as f:
    version = pyjapt.__version__

# After
version = pyjapt.__version__

🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

…ctly

Co-authored-by: alejandroklever <45394625+alejandroklever@users.noreply.github.com>
Copilot AI changed the title [WIP] Address feedback on create lalr grammar tests PR Remove redundant file open in build.py Feb 28, 2026
@alejandroklever alejandroklever marked this pull request as ready for review March 3, 2026 17:03
@alejandroklever alejandroklever merged commit 47d9e40 into feature/implement-tests Mar 3, 2026
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.

2 participants