search

bachors / jQuery-Awesome-Sosmed-Share-Button

Awesome Social Media Share Button With Share Count
http://bachors.github.io/jQuery-Awesome-Sosmed-Share-Button/
MIT License
58 stars 35 forks source link

Penambahan test dan npm #12

Closed riochr17 closed 7 years ago

riochr17 commented 7 years ago

Penambahan

Beberapa hal yang ditambahkan pada pull request ini

  1. Node package manager file (package.json)
  2. Node modules (berikutnya akan di ignore)
  3. .travis.yml (automatic check + build test)
  4. direktori test (berisi index.js dan line.html)

    Perubahan

Perubahan dilakukan pada src/js/ayoshare.js, perubahan hanya diseputar semantik dan syntax yang disesuaikan dengan standar kualitas jshint. Bagian ini tidak mengubah fungsionalitas sama sekali.

Pengujian

Pada package.json terdapat script test melakukan pengujian sbb:

test/index.js merupakan script uji untuk menguji fungsionalitas src/js/ayoshare.js, pengujian dilakukan menggunakan data line.html sebagai html percobaan. Pengujian dilakukan menggunakan mocha-chai framework test.

Pre-commit

Dependency pre-commit juga ditambahkan pada pull request ini. Digunakan supaya pengujian otomatis dilakukan setiap kali melakukan commit. Jika pengujian gagal, commit tidak akan dilakukan.

riochr17 commented 7 years ago

done @jayvdb, thanks for suggestion.

bachors commented 7 years ago

Gimana mas @riochr17 ko ga bisa di merge ya ?

riochr17 commented 7 years ago

@bachors ada konflik di src/js/ayoshare.js harus di resolved dulu.

kalau di konsol terminal, pakai git mergetool

atau biar saya perbaiki konfliknya saja, mohon tunggu sebentar

riochr17 commented 7 years ago

conflict resolved, done @bachors , silahkan di merge

riochr17 commented 7 years ago

@jayvdb is it a problem if i revert back cafb6fd821eea2c58ff2eef20ab443a025b0aa49 commit?

bachors commented 7 years ago

Sip sudah mas @riochr17 trimakasih banyak nambah ilmu baru 👍 kebetulan saya masih kurang faham cara penggunaan github.

riochr17 commented 7 years ago

Sama-sama @bachors

btw, pada PR ini saya mengasumsikan mas @bachors sudah terbiasa dengan npm (at least package.json) jika blm bisa cari di google e.g. tutorialspoint.

mas @bachors harus memastikan memiliki npm pada lokal (konsol) Anda, jika tidak mungkin commit akan gagal.

Saya juga menambahkan automatic build + test menggunakan Travis CI (berkas .travis.yml), mungkin mas @bachors tertarik memasang Travis CI pada repo ini. Contoh repo menggunakan Travis CI klik disini

Pada PR ini saya menggunakan Ubuntu 16.04, jika terdapat kesalahan pada lokal mas @bachors setelah merge PR ini silahkan di revert ke checkpoint terakhir sebelum merge ini atau mungkin bisa kita diskusikan email saya riochr17@gmail.com

Saya sangat senang bisa berdiskusi dengan Anda mas @bachors :)

regards