Skip to content

Removed Alert of LGTM for Graph.hpp - #142

Merged
ZigRazor merged 1 commit into
masterfrom
LGTM-Alert-Remove
Oct 21, 2021
Merged

ZigRazor merged 1 commit into
masterfrom
LGTM-Alert-Remove

Conversation

@ZigRazor

Copy link
Copy Markdown
Owner

No description provided.

@ZigRazor ZigRazor added enhancement New feature or request repo something about repo labels Oct 21, 2021
@github-actions github-actions Bot added core something about core and removed repo something about repo labels Oct 21, 2021
@ghost

ghost commented Oct 21, 2021

Copy link
Copy Markdown

Changes to your CodeSee Architecture Map:

View a CodeSee Map of these changes

View this PR diagram as an interactive map

View more CodeSee Maps

Legend

CodeSee Map Legend

@lgtm-com

lgtm-com Bot commented Oct 21, 2021

Copy link
Copy Markdown

This pull request fixes 4 alerts when merging ab201e8 into c80ec1d - view on LGTM.com

fixed alerts:

  • 2 for Constant return type on member
  • 1 for Non-virtual destructor in base class
  • 1 for Declaration hides parameter

@ZigRazor
ZigRazor merged commit 6fb620a into master Oct 21, 2021
@ZigRazor
ZigRazor deleted the LGTM-Alert-Remove branch October 21, 2021 07:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core something about core enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant