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

feat: replace get and set methods with unified value option #232

Merged
merged 11 commits into from
May 21, 2024

Conversation

smalluban
Copy link
Contributor

@smalluban smalluban commented Jan 29, 2024

BREAKING CHANGE:

  • get and set methods are replaced with value option
  • Properties no longer reflect value to the corresponding attribute by default - the new reflect option must be used

Copy link

codesandbox-ci bot commented Jan 29, 2024

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

@smalluban smalluban changed the title Replace get and set methods with unified value option feat: replace get and set methods with unified value option Jan 29, 2024
@smalluban smalluban marked this pull request as draft January 29, 2024 10:59
@coveralls
Copy link

coveralls commented Feb 16, 2024

Coverage Status

coverage: 99.956% (+0.05%) from 99.911%
when pulling 07e4f3c on feat-value-over-get-set
into 3208959 on main.

BREAKING CHANGE: `get` and `set` methods must be replaced  with `value` option. Properties no longer
reflect value to the corresponding attribute by default. New `reflect` option must be set to `true`.
@smalluban smalluban marked this pull request as ready for review May 17, 2024 07:34
@smalluban smalluban merged commit 258e41f into main May 21, 2024
3 checks passed
@smalluban smalluban deleted the feat-value-over-get-set branch May 21, 2024 07:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants