dgraph-io / dgraph-js

Official Dgraph JavaScript client
Apache License 2.0
323 stars 52 forks source link

chore(Github Action|CI): add Github Action for testing #169

Closed MichelDiz closed 1 year ago

MichelDiz commented 1 year ago

Reusing existing test scripts to bring CI into the repository.


This change is Reviewable

skrdgraph commented 1 year ago

@joshua-goldstein / @dshekhar95 could you also take a look .. I am starting to review this. If you see any issues pls flag it here. I know you both are fixing other clients as well (hence adding you)

MichelDiz commented 1 year ago

@skrdgraph

Good, all points we should address with more discussion. Standarize Node is really good, but hard. Some repos we use legacy code, so we need to review then and replace if necessary. Ratel is an example, it is painful to make any progress there.

Change it to main we will need to check the side effects. If you have something in another repository being master instead of main can break things. So we need more ppl looking into it.

coverage, watchAll, CI, testresultsprocessor. Again, other stuff we can add to Jira and work on.