Documentation improvements in the code base.
@return type
@throws
@param
values in the comment docblocks.

CommentFileSizeAuthor
#7 2608880-7.patch2.66 KBrenatog
#4 2608880-4.patch2.98 KBxaiwant
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

heykarthikwithu created an issue. See original summary.

naveenvalecha’s picture

naveenvalecha
He/Him
Primary language Hindi
CreditAttribution: naveenvalecha as a volunteer commented
Issue summary: View changes
xaiwant’s picture

xaiwant CreditAttribution: xaiwant as a volunteer and at Valuebound commented
Assigned: heykarthikwithu » xaiwant
Status: Active » Needs work

Working on the documentation improvements, adding comments in code and code cleanup.

xaiwant’s picture

xaiwant CreditAttribution: xaiwant as a volunteer and at Valuebound commented
Assigned: xaiwant » Unassigned
Status: Needs work » Needs review
FileSize
2608880-4.patch2.98 KB

Added patch for Document changes.

heykarthikwithu’s picture

heykarthikwithu
Primary language English
Location Bengaluru 🌍
CreditAttribution: heykarthikwithu as a volunteer and at Valuebound commented
Issue tags: +Documentation
heykarthikwithu’s picture

heykarthikwithu
Primary language English
Location Bengaluru 🌍
CreditAttribution: heykarthikwithu as a volunteer and at Valuebound commented
Title: Documentation improvements in the code base. » Add @return type, @throws and @param values in the comment docblocks.
Issue summary: View changes
Issue tags: +Quick fix
renatog’s picture

renatog
Location Campinas
CreditAttribution: renatog at CI&T commented
FileSize
2608880-7.patch2.66 KB

This update really makes sense. The documentation is very important to have the code more organized and easier to read. This is the patch updated

  • RenatoG committed fb43cb1 on 7.x-1.x authored by xaiwant 
    Issue #2608880 by xaiwant, RenatoG, heykarthikwithu, naveenvalecha: Add...
renatog’s picture

renatog
Location Campinas
CreditAttribution: renatog at CI&T commented
Status: Needs review » Fixed

Moved to the dev branch

Thank you so much @xaiwant and everyone for your contribution

Great job

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.