Closed khushishikhu closed 4 years ago
@SanketDG please review my changes.
Merging #662 into develop will not change coverage. The diff coverage is
n/a
.
@@ Coverage Diff @@
## develop #662 +/- ##
========================================
Coverage 89.49% 89.49%
========================================
Files 49 49
Lines 2693 2693
========================================
Hits 2410 2410
Misses 283 283
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update 0aa1f2f...91240ca. Read the comment docs.
Make sure to follow the commit message guidelines
Make sure to follow the commit message guidelines
yeah sure. Thank you.
You will need to change the commit message, not the PR title.
@SanketDG i have changed the commit message style. can you please review it . if it's okay or not now.
@SanketDG Thank you for your approval!!
@saideepesh000 can you please review my changes.
@saideepesh000 can you please review my changes.
@khushishikhu, thanks for mentioning me, actually I am not the mentor or maintainer of this project, thanks.
@saideepesh000
Anyone can review a PR, you can too, it would be of great help!
Whether it's merged or not is upon the maintainers.
@saideepesh000 okay, thank you.
@sammy1997 can you please review my changes.
@sakshi1499 Thank you for your approval !!
Description Changed in code so that virtual environment can activate correctly.
Fixes #591
Type of Change:
Checklist: