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

Refactoring #1154

Merged

ota-meshi merged 2 commits into master from refactor-no-side-effects-in-computed-properties

May 21, 2020

Member

ota-meshi commented May 21, 2020

Made the side effect analysis logic of the "no-side-effects-in-computed-properties" rule common.
Made the setup function visit common.
Made the Vue object visit common.

ota-meshi added 2 commits

May 21, 2020 22:55


          Refactoring

2f73e8c


          update

e18a886

ota-meshi merged commit e80c2f0 into master

ota-meshi deleted the refactor-no-side-effects-in-computed-properties branch

May 21, 2020 21:14

ota-meshi mentioned this pull request

no-side-effects-in-computed false positives #1282

Closed

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