Hi Thierry,

I will start from "PHP syntax check 
improvement"(https://github.com/ibazzi/pdt/pull/3)


Recently I have not enough time to add unit tests.


But I will learn how to use gerrit first.


------------------ 原始邮件 ------------------
发件人: "thierry blind";<[email protected]>;
发送时间: 2017年3月23日(星期四) 凌晨3:37
收件人: "PDT Developers"<[email protected]>; 

主题: Re: [pdt-dev] Introduce new PHP Server plugin



  
Hi ibazzi,
 
 
I feel you will become a great PDT contributor ;)
 
You made a lot of patches, it would be great to include them in our repository 
so everybody would benefit from your work ;)
 

 
 
The best way in my opinion is still to submit them through gerrit, so that we 
can review them quietly. First step would be (on your side) to priorize the 
most important patches you'd like to send us ;)
 
For real bugfixes (or new features) a bug report should be open + you should 
also add some unit tests to the patches you'll post on gerrit. It adds a bit 
work, but it will help us to understand your patches and also avoid regressions.
 

 
 
For a minor change, a "No bug" patch will be enough.
 

 
 
In all cases, don't worry, we'll make reviews and comments in gerrit to help 
you for your first contributions ;)
 
 
 
If necessary, you also have this little guide to start with :  
https://wiki.eclipse.org/PDT/Contributing
 

 
 
Thierry.
 

 
  
 De : [email protected] <[email protected]> de la part de 
ibazzi <[email protected]>
 Envoyé : mardi 21 mars 2017 15:00
 À : PDT Developers
 Objet : Re: [pdt-dev] Introduce new PHP Server plugin  
 
  oh, sure, you are welcome.
 
 
 Let's make it better.
  
 
 BTW, this my pdt repository with serval enhancement(CA, syntax validator, 
quick fix, debug, php search, organize use statements, etc)
 https://github.com/ibazzi/pdt
 https://github.com/ibazzi/pdt/pulls?q=is%3Apr+is%3Aclosed
 https://github.com/ibazzi/dltk.core
 
 
 let's see which can be merged into PDT main repository.
 
 
 ------------------ Original ------------------
  From:  "Michal Niewrzal";<[email protected]>;
 Date:  Tue, Mar 21, 2017 10:41 PM
 To:  "[email protected]"<[email protected]>; 
 
 Subject:  [pdt-dev] Introduce new PHP Server plugin
 
 
 
  
Hi,
 

 
 
Yes, your plugin looks very interesting :) I was able to test it but to do this 
I had to make some hacks in your code. If you like I can open GitHub issue to 
describe main problems.
 

 
 
Regards,
   

 
 
 
 Michał Niewrzał | ZEND STUDIO
 
 Rogue Wave Software, Inc.
 
 Accelerating Great Code
 

 
 
 
 www.roguewave.com /  [email protected]
 

 
 
 
 
  
 Od: [email protected] <[email protected]> w imieniu 
użytkownika Kaloyan Raev <[email protected]>
 Wysłane: 21 marca 2017 15:27
 Do: [email protected]
 Temat: Re: [pdt-dev] Introduce new PHP Server plugin  
 
 Hi,
 
 This looks nice! It actually addresses an enhancement bug we have in PDT 
Bugzilla:  https://bugs.eclipse.org/bugs/show_bug.cgi?id=440485
 
 It would be great if you do extra mile and contribute your work to PDT. You 
can consult this page for a contribution guide:  
https://wiki.eclipse.org/PDT/Contributing
 
 Your first goal would be to push a Gerrit patch with your work that we can 
review. Let us know if you face any troubles.
 
 --
 Kaloyan Raev | Zend Studio Team Lead
 Rogue Wave Software, Inc.
 Accelerating Great Code
 M +359 887 648 663
 www.roguewave.com / [email protected] 
 
 
 On 03/21/2017 03:11 PM, ibazzi wrote:
 
  Hi there,
 
 
 I have implemented the wtp server adapter for PHP with php built-in server. 
See the attachment.
 
 
 Now we can add and publish projects, and debug with php built-in server.
 
 
 Here is the respository for the plugin, and also some changes in pdt(work in 
progress).
 
 
 https://github.com/ibazzi/phpserver
 https://github.com/ibazzi/pdt/tree/phpserver
 
 
 If you are instrested in it, please let me know.
 
 
 Best regards
 
  
 _______________________________________________ pdt-dev mailing list 
[email protected] To change your delivery options, retrieve your password, or 
unsubscribe from this list, visit 
https://dev.eclipse.org/mailman/listinfo/pdt-dev
_______________________________________________
pdt-dev mailing list
[email protected]
To change your delivery options, retrieve your password, or unsubscribe from 
this list, visit
https://dev.eclipse.org/mailman/listinfo/pdt-dev

Reply via email to