« Back Issue #256
  • 2015-04-02 18:27 Reported by niravkumar
  • 2015-04-12 22:54 Marked fixed
Status: Marked fixed
Priority: Medium

Portuguese Stemmer

niravkumar 2015-04-02 18:27

Adding new module to be used to stem Portuguese words

Steps to reproduce: NA

Additional information: NA

administrator 2015-04-02 22:25

PLease upload your patch when ready

administrator 2015-04-03 07:26

Hi Nirav,

Thanks for your patch. Please revise it addressing the issues below: (1) Your patch has a lot of tabs rather than 4 white spaces in it, and also stray white space at the end of lines (2) You have a few lines over 80 chracters, for example, around line 162 in the patch. (3) Several of your variable names are not following Yioop coding conventions that variables should be all lower case with words separated by underscores. For example, variables like $SpecialSuffix should be $special_suffix.

Best, Chris

niravkumar 2015-04-09 21:22

Hi Chris,

Thanks for the update. I have attached updated patch.

Please have look and let me know if further change required.

Thanks, Nirav

administrator 2015-04-12 22:53

Hey Nirav,

Thanks for your patch. It is now live.

Best, Chris

administrator 2015-04-12 22:54

Fixed by 69dfc15d1

niravkumar 2015-04-12 22:54
X