Update graph ops parser & cleanup

alifahrri commented 3 years ago

Type of changes

Please check the type of change your PR introduces: - [x] Bugfix - [ ] Feature - [ ] Code style update (formatting, renaming) - [ ] Refactoring (no functional changes, no api changes) - [ ] Build related changes - [ ] Documentation content changes - [ ] Other (please describe): ## What is the current behavior?

graph ops using third party apps to do parsing

Related issues :

What is the new behavior?

graph ops using builtin json to do parsing

Checklist

[x] I have added tests that prove my fix is effective or that my feature works
[x] I have added necessary documentation (if appropriate)
[x] Any dependent changes have been merged and published in downstream modules
[x] I have filled the [Unreleased] Changelog

triwahyuu commented 3 years ago

for the CI, I think it is better to separate the linter and testing, since testing only have been set up, probably you could delete the testing phase in "Lint and Test" action.

triwahyuu commented 3 years ago

other than that, LGTM

alifahrri commented 3 years ago

thanks @triwahyuu

nodefluxio / vortex