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.

Sign up for GitHub

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

Jump to bottom

Merge auto-fields-in-methods into main: Remove need for this on class vars #225

Merged

stevehalliwell merged 1 commit into main from auto-fields-in-methods

Dec 16, 2023

Owner

stevehalliwell commented Dec 15, 2023

Since moving to entirely frozen classes, we can now remove the need for this when accessing a field on the instance in a method.

Remember to Add/Update the:

Sample scripts
Test scripts


          Add get set Field

2795fcf

allows for methods to auto this.

stevehalliwell linked an issue

that may be closed by this pull request

Remove need for explicit 'this' when vars are used. #20

Closed

github-actions bot commented Dec 15, 2023

coverage: 88.469% (-0.007%) from 88.476%
when pulling 2795fcf on auto-fields-in-methods
into cb62014 on main.

stevehalliwell merged commit 19000cb into main

4 checks passed

stevehalliwell deleted the auto-fields-in-methods branch

December 16, 2023 02:48

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet