search

ETDA / e-TaxInvoice-PDFgen

e-Tax Invoice by Email - PDF Generator on PC
GNU Affero General Public License v3.0
134 stars 89 forks source link

add back csproj #7

Closed varokas closed 7 years ago

varokas commented 7 years ago

From this commit https://github.com/ETDA/e-TaxInvoice-PDFgen/commit/b226052ceb5cd6cdfd8cb5d88f9c9e47c5554d3b I think you delete csproj by mistake

varokas commented 7 years ago

Also please consider making changes using pull requests rather than push directly to master. It's pretty hard to tell what changed for what reason.

ultimaweapon commented 7 years ago

I don't think that will work for now since it needs more developers to be able to do code review. Currently, from git histories, it seems like there is only one developer in their organization. I think we can do that after:

  1. We have more active contributors.
  2. Or their organization have more developers working on this project.

BTW, why we talk in English? In my opinion, we should talk in Thai. Because if we talk in English, we might scare other Thai developers to join this project :). It is good to practice English, but we should welcome Thai developers to join this project to learn how open source work and make them comfortable before practice English.

pragasitcha commented 7 years ago

@varokas Also please consider making changes using pull requests rather than push directly to master. It's pretty hard to tell what changed for what reason.

ขอขอบุณคุณ @varokas และคุณ @ExUltima มาก ครับขออนุญาตถาม คุณ @varokas เป็นภาษาไทยครับ จากข้อความภายใน quote นั้นหมายความว่า ผมควรจะ push directly ไปที่ master เอง เพื่อที่จะสามารถติดตามได้ว่าตัว code ถูกเปลี่ยนเนื่องมาจากจากอะไรและเพื่ออะไรควรต้องออกมาในลักษณะแบบนี้หรือเปล่าครับ

aixasz commented 7 years ago

@pragasitcha ไม่ใช่ครับ

ผมขอตอบเป็นภาษาไทยเพื่อให้เข้าใจง่ายขึ้น และเข้าใจตรงกันครับ จากที่คุณ @varokas กล่าวมาคือ

ทาง contributor หรือคุณ @pragasitcha ได้มีการเปลี่ยนแปลง code แล้วทำการ push ตรงไปยัง master เลยนั้น ทำให้ติดตามยากว่า changes ที่เกิดขึ้นในแต่ละ commit ทำไปเพื่ออะไร และทำให้คนอื่นที่เข้ามาช่วย contribute ไม่รู้ว่า changes ที่ push เข้าไป ทำอะไร เพื่ออะไร และมีอะไรต้องปรับปรุงหรือไม่

ดังนั้นทางที่ดีควรจะสร้าง branch ขึ้นมาจาก master แล้วก็ทำงานบน branch นั้นจนเสร็จ จึงค่อย commit และ push เข้า remote ของตัวเอง หลังจากนั้นจึงทำการสร้าง pull request เพื่อให้ developer หรือ contributor ท่านอื่นเข้ามาทำ code review ได้ครับ

เสริมครับ

ultimaweapon commented 7 years ago

@varokas ไม่มีความเห็นอะไรหน่อยเหรอครับ

varokas commented 7 years ago

ไม่มีอะไรเพิ่มเติมครับ