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

Cleanup expression usage #750

Merged
merged 16 commits into from
Aug 22, 2023
Merged

Cleanup expression usage #750

merged 16 commits into from
Aug 22, 2023

Conversation

terrorfisch
Copy link
Member

@terrorfisch terrorfisch commented Jan 17, 2023

Use evaluate_in_scope where appropriate.

  • Newspiece with changes
  • Conceptional documentation in expression/__init__.py
  • Tests that only methods covered by the Protocol are used
  • Adjust documentation

Postponed todos are here #785

@github-actions
Copy link

github-actions bot commented Jan 17, 2023

Test Results

       6 files         6 suites   6m 25s ⏱️
1 216 tests 1 161 ✔️   55 💤 0
7 296 runs  6 966 ✔️ 330 💤 0

Results for commit cb432af.

♻️ This comment has been updated with latest results.

@terrorfisch terrorfisch mentioned this pull request Mar 28, 2023
3 tasks
@terrorfisch terrorfisch marked this pull request as ready for review August 22, 2023 07:38
@terrorfisch terrorfisch merged commit 3d5d96d into master Aug 22, 2023
9 checks passed
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.

1 participant