EnvisEdge icon indicating copy to clipboard operation
EnvisEdge copied to clipboard

updated data_module folder

Open haripriya9647 opened this issue 2 years ago • 4 comments

Description

adding inline doc for all the classes and methods

Relevant Issue

Issue #276 the issue here

Affected Dependencies

List any dependencies that are required for this change.

How has this been tested?

  • Describe the tests that you ran to verify your changes.
  • Provide instructions so we can reproduce.
  • List any relevant details for your test configuration.

Checklist

haripriya9647 avatar Jul 06 '22 17:07 haripriya9647

Deploy Preview for goofy-agnesi-4565cb canceled.

Name Link
Latest commit 7b17f46f5ac0bcd2b0d607915ed6307b5e6a90eb
Latest deploy log https://app.netlify.com/sites/goofy-agnesi-4565cb/deploys/630768bd1d6e050008912c2d

netlify[bot] avatar Jul 06 '22 17:07 netlify[bot]

@haripriya9647 In this PR, please try to write the whole inline documentation inside the class. You can explain about methods inside it. Only write inline docs for methods if the code flow of the method is complex.

divyanshugit avatar Jul 07 '22 11:07 divyanshugit

Please utilize the 70 letters in each file.

divyanshugit avatar Aug 25 '22 08:08 divyanshugit

@divyanshugit i have added commits as per suggestions.Please review

haripriya9647 avatar Aug 25 '22 12:08 haripriya9647