Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Development #52

Merged
merged 13 commits into from
Oct 14, 2021
Merged

Development #52

merged 13 commits into from
Oct 14, 2021

Conversation

pranjalchanda08
Copy link
Contributor

Add latest changes to stable branch.

akashkollipara and others added 13 commits October 3, 2021 00:37
@akashkollipara
<Jenkins> Added TC repo path
22a47b0
@akashkollipara
<LibNMath> Added new function to neomath library
9e2a267
@akashkollipara
<Jenkins> minor update in pipeline script
dcf5330
@pranjalchanda08
Merge pull request #41 from VisorFolks/dev_akashkollipara_nmath_update
ae027b4 
<LibNMath> Added new function to neomath library
@akashkollipara
<Lint> Adding linting tool support "cppcheck"
497db79 
- Added a new target 'check' to perform linting on project cpde base
- cppcheck package check is added
- Tested on existing code base
@akashkollipara
Added a small fix for cpp check on constructors
c499801
@akashkollipara
minor fixes for error detected by cppcheck
d7c7f5b
@akashkollipara
Updated jenkins flow to perform linting
3e89777
@akashkollipara
minor fix for supressing warnings
50736cd
@MayuriLokhande
Bug fix: Replaced ARC with CC
886f21a
@akashkollipara
Merge pull request #51 from VisorFolks/dev_mayurilokhande_bugfix
43d61ba 
Bug fix: Replaced ARC with CC
@akashkollipara
Merge branch 'development' into dev_akashkollipara_feat_lint
a763a21
@pranjalchanda08
Merge pull request #43 from VisorFolks/dev_akashkollipara_feat_lint
a933849 
Dev akashkollipara feat lint
@akashkollipara akashkollipara requested a review from a team October 14, 2021 16:11
@akashkollipara akashkollipara added bug-critical System failure can be predicted critical update This is a critical update cyancore framework Cyancore framework feature request enhancement Upgrading feature or request FR Add new feature or request terravisor Terravisor related issue labels Oct 14, 2021
@akashkollipara akashkollipara added this to the Release #2 milestone Oct 14, 2021
This was linked to issues Oct 14, 2021
FR: Add linting tool support #44
Closed
Correction of variable name in makefile #46
Closed
akashkollipara
akashkollipara approved these changes Oct 14, 2021
View reviewed changes
Copy link
Member

@akashkollipara akashkollipara left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd request other @VisorFolks/cc-admin to have a look...

@akashkollipara akashkollipara requested review from a team and PoojaIngalagi October 14, 2021 16:16
@PoojaIngalagi PoojaIngalagi merged commit 44314aa into stable Oct 14, 2021
@pranjalchanda08
Copy link
Contributor Author

Congrats!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@akashkollipara akashkollipara akashkollipara approved these changes

@PoojaIngalagi PoojaIngalagi PoojaIngalagi approved these changes

@shreeroopajay shreeroopajay Awaiting requested review from shreeroopajay

Assignees

@akashkollipara akashkollipara

@pranjalchanda08 pranjalchanda08

@MayuriLokhande MayuriLokhande

Labels
bug-critical System failure can be predicted critical update This is a critical update cyancore framework Cyancore framework feature request enhancement Upgrading feature or request FR Add new feature or request terravisor Terravisor related issue
Projects
Terravisor
Status: Done
Milestone
Helium v1.2.0
Development

Successfully merging this pull request may close these issues.

Correction of variable name in makefile FR: Add linting tool support
4 participants
@pranjalchanda08 @akashkollipara @PoojaIngalagi @MayuriLokhande