Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Python File Concept #2100

Open
wants to merge 150 commits into
base: main
Choose a base branch
from
Open

Python File Concept #2100

wants to merge 150 commits into from

Conversation

maximiliankaul
Copy link
Contributor

@maximiliankaul maximiliankaul commented Mar 3, 2025

Implementation of file concept and operation nodes for Pythons builtin open and os.open.

@maximiliankaul
Copy link
Contributor Author

#2118 should be merged here, before this pr is merged into main.

@oxisto oxisto modified the milestones: 9.4, 10.x Mar 11, 2025
): T? {
val arg =
name?.let { this.argumentEdges[it]?.end } ?: position?.let { this.arguments.getOrNull(it) }
val value = evaluator.evaluateAs<T>(arg)
val value = this.language.evaluator.evaluateAs<T>(arg)
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This works. I'd prefer to have this configurable, but a parameter with default value doesn't work:

evaluator: ValueEvaluator = this.language.evaluator

@maximiliankaul maximiliankaul requested a review from peckto as a code owner March 12, 2025 11:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
concepts Related to CPG concepts module publish-to-github-packages If added to a PR, builds from it will be published as a GitHub package python
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants