Skip to content
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

[Fixed #23] Fix all eslint syntax and style issues #28

Merged
merged 5 commits into from
May 24, 2016
Merged

[Fixed #23] Fix all eslint syntax and style issues #28

merged 5 commits into from
May 24, 2016

Conversation

gngchrs
Copy link
Contributor

@gngchrs gngchrs commented May 24, 2016

No description provided.

kn9ts added 3 commits May 21, 2016 00:03
* develop: (27 commits)
  Added CURL example to README
  Added missing mongodb installation steps
  Renamed http_code to status_code
  Replaced image with one delivered from js.co.ke
  Updated banner
  Added banner image to readme
  Adding project banner
  Finished #17 Add installations steps to README
  Finished #22 Add field/params validation to /payment/request route
  Finished #10 Refactor ES6 import and any ES2015 feature not supported by Argon (Node.js v4.4.4.)
  Updated express and some of dependencies
  Updated /payment/request paymentDetails to use underscore notation on response
  Updated response JSON example in README
  Finshed #19 Wrap successful responses in response object
  Finished #20 Add sample request and response to README
  Finished #20 Add sample request and response to README
  Made /payment/request a POST request
  Replaced include method with indexOf while checking for extraParams
  Added payment request details to JSON response
  Added missing return code: 43
  ...
* master:
  Updated README
  Removed an dependencies and usage of mongodb in the app, not really required at the moment
@roboflank
Copy link

@gangachris @kn9ts Travis build seems to be failing, had you configured it?
Though it's a non issue.

@kn9ts
Copy link
Owner

kn9ts commented May 24, 2016

@denzelwamburu I haven't yet. I'm still considering using it. My preference is SemaphoreCI

@kn9ts kn9ts added the chore label May 24, 2016
@kn9ts kn9ts added this to the v1.0.0 milestone May 24, 2016
@roboflank
Copy link

@kn9ts I haven't used before SemaphoreCI but i Travis works great with open source, you can give it a try 😄

@roboflank
Copy link

roboflank commented May 24, 2016

I have Created a Pull for Travis in case you consider using it #29

@kn9ts kn9ts changed the title [Fixed #23] Fix all eslint syntax issues [Fixed #23] Fix all eslint syntax and style issues May 24, 2016
@kn9ts kn9ts merged commit 7347149 into kn9ts:develop May 24, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants