Skip to content

remove LGTM#292

Merged
DaveSkender merged 5 commits intoupdate-packagesfrom
codeql
Dec 27, 2022
Merged

remove LGTM#292
DaveSkender merged 5 commits intoupdate-packagesfrom
codeql

Conversation

@DaveSkender
Copy link
Member

@DaveSkender DaveSkender commented Nov 13, 2022

Description

LGTM is deprecated and is now part of GitHub CodeQL.

To do

  • add CodeQL actions
  • remove LGTM hooks on their site
  • remove issues bot action (unrelated)

Checklist

  • My code follows the existing style, code structure, and naming taxonomy
  • I have commented my code, particularly in hard-to-understand areas
  • I have performed a self-review of my own code and included any verifying manual calculations
  • I have added or updated unit tests that prove my fix is effective or that my feature works, and achieves sufficient code coverage. New and existing unit tests pass locally and in the build (below) with my changes
  • My changes generate no new warnings and running code analysis does not produce any issues
  • I have added or run the performance tests that depict optimal execution times
  • I have made corresponding changes to the documentation

@DaveSkender DaveSkender changed the base branch from main to update-packages December 27, 2022 18:08
@DaveSkender DaveSkender changed the title add CodeQL remove LGTM Dec 27, 2022
@DaveSkender DaveSkender marked this pull request as ready for review December 27, 2022 18:09
@DaveSkender DaveSkender merged commit 452875e into update-packages Dec 27, 2022
@DaveSkender DaveSkender deleted the codeql branch December 27, 2022 18:10
DaveSkender added a commit that referenced this pull request Dec 30, 2022
* update to core DLL v2.4.7
* update doc site packages
* fix URLs to C# code
* remove LGTM (#292)
* remove issues bot
* remove obsolete badges
* update dependabot
* remove obsolete v0.8 branch trigger
* update unit tests
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 1, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant