-
Notifications
You must be signed in to change notification settings - Fork 94
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
Added treeitem role to tree items. Updated a few devDependencies. #182
Conversation
Second commit updating all the possible dependencies without breaking the build. The only dependency that wasn't updated was If you don't want to merge this one, at least merge the first one that was just a small change. |
I've updated eslint and introduced vue-cli-service. As a suggestion you could try and convert the entire project to vue-cli-service and use it instead of rollup for a version 0.3. Please check the commits (and don't mind the broken .eslintrc.js on the 3rd commit, I fixed it on the 4th) and see if it's something useful for the project and merge it. With this I'll close #181 now. Edit: You should really update your tests add more. The code is barely tested. Especially the asynchronous behaviour and the RequestFrameAnimation part. |
Just saw PR #183 that was a lot more elegant fix than my way to get rid of the errors I mention in the 3rd commit. |
@amsik could you review the pull requests? |
@amsik done. |
Here's the pull request with what I was talking about in #181 issue. Could you merge and release as 0.2.70?