Flovilmart/readme updates

[flovilmart]

No description provided.

[codecov]

Codecov Report

Merging #588 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #588   +/-   ##
=======================================
  Coverage   84.57%   84.57%           
=======================================
  Files          48       48           
  Lines        4039     4039           
  Branches      911      911           
=======================================
  Hits         3416     3416           
  Misses        623      623

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update de4bc1c...80e095b. Read the comment docs.

[dplewis]

Can you update the Contribution.MD? With instruction for Unit test, integration test, jest etc. After your improvements with the parse-server repo I feel like I’ve been testing wrong here

[flovilmart]

I'm not sure what need to be tested on this repo. Can you try to update it (the CONTRIBUTING) and let me review?

[dplewis]

My English is really bad, plus I have my own method of testing this repo when I do PR’s. I’m curious on how you do it. Also it’s very outdated (Contribution.MD)

[flovilmart]

plus I have my own method of testing this repo when I do PR’

How do you do it?

[dplewis]

@montymxb parse-server-test script + multiple terminal windows on my Mac + there’s no fit for mocha so I have to comment out a lot of the test

[flovilmart]

Oh wow! That’s not an optimized workflow at all :) do you use VSCODE?

[dplewis]

Nope, I’ve seen developers use it over time but never picked it up. I’ve never once set a breakpoint using parse since I started contributing

[flovilmart]

Haha :$ try the new contributing guide on parse-server then and tell me what’s broken :)

[dplewis]

I got you, but updated it here also please 🙏

Bring it to 2018

[flovilmart]

Hahaha i’ll Try :)

[flovilmart]

@dplewis for the sake of consistency, I may update the tests from mocha to jasmine for the integration.

[dplewis]

Speaking of outdated I know we want to deprecate this

https://github.com/parse-community/parse-server-example

When I get a new client I refer to this but it’s still using Node 4.3 instead of 8. (I should be using CLI I know) Can I have access to it?

[dplewis]

@flovilmart npm run test + console.log is all I know

[dplewis]

@flovilmart Thanks a lot, I was surprise there was no issues tab, we could catch regression if it was enabled.

parse-server-example