Skip to content

TranslationProject - #3

Draft
gruenwaldlk wants to merge 20 commits into
masterfrom
feature/migrate_translation_project
Draft

TranslationProject#3
gruenwaldlk wants to merge 20 commits into
masterfrom
feature/migrate_translation_project

Conversation

@gruenwaldlk

Copy link
Copy Markdown
Owner

Implements "Translation Projects":

  • Project creation
  • Project migration

Kad Venku added 4 commits February 11, 2020 15:22
Updates log handling, adds various new utilities and test cases.
Text projects can be created from the base game translations.
Supported types:
  * CSV
  * NLS
@gruenwaldlk
gruenwaldlk force-pushed the feature/migrate_translation_project branch from a3565b2 to 23169a9 Compare February 11, 2020 14:38
@lgtm-com

lgtm-com Bot commented Feb 11, 2020

Copy link
Copy Markdown

This pull request introduces 2 alerts when merging 23169a9 into 0912668 - view on LGTM.com

new alerts:

  • 1 for Dereferenced variable may be null
  • 1 for Missing Dispose call on local IDisposable

@lgtm-com

lgtm-com Bot commented Feb 11, 2020

Copy link
Copy Markdown

This pull request introduces 1 alert when merging f277401 into 0912668 - view on LGTM.com

new alerts:

  • 1 for Dereferenced variable may be null

@lgtm-com

lgtm-com Bot commented Feb 11, 2020

Copy link
Copy Markdown

This pull request introduces 1 alert when merging 8828985 into 0912668 - view on LGTM.com

new alerts:

  • 1 for Dereferenced variable may be null

@lgtm-com

lgtm-com Bot commented Feb 11, 2020

Copy link
Copy Markdown

This pull request introduces 1 alert when merging d480c47 into 0912668 - view on LGTM.com

new alerts:

  • 1 for Dereferenced variable may be null

@lgtm-com

lgtm-com Bot commented Feb 11, 2020

Copy link
Copy Markdown

This pull request introduces 1 alert when merging 4d87b22 into 0912668 - view on LGTM.com

new alerts:

  • 1 for Dereferenced variable may be null

@lgtm-com

lgtm-com Bot commented Feb 11, 2020

Copy link
Copy Markdown

This pull request introduces 1 alert when merging 62ae789 into 0912668 - view on LGTM.com

new alerts:

  • 1 for Dereferenced variable may be null

@lgtm-com

lgtm-com Bot commented Feb 11, 2020

Copy link
Copy Markdown

This pull request introduces 1 alert when merging 223c359 into 0912668 - view on LGTM.com

new alerts:

  • 1 for Dereferenced variable may be null

@lgtm-com

lgtm-com Bot commented Feb 11, 2020

Copy link
Copy Markdown

This pull request introduces 1 alert when merging a007330 into 0912668 - view on LGTM.com

new alerts:

  • 1 for Dereferenced variable may be null

@lgtm-com

lgtm-com Bot commented Feb 11, 2020

Copy link
Copy Markdown

This pull request introduces 1 alert when merging bb750d8 into 0912668 - view on LGTM.com

new alerts:

  • 1 for Dereferenced variable may be null

@lgtm-com

lgtm-com Bot commented Feb 11, 2020

Copy link
Copy Markdown

This pull request introduces 1 alert when merging e3690cf into 0912668 - view on LGTM.com

new alerts:

  • 1 for Dereferenced variable may be null

@lgtm-com

lgtm-com Bot commented Feb 11, 2020

Copy link
Copy Markdown

This pull request introduces 1 alert when merging b250047 into 0912668 - view on LGTM.com

new alerts:

  • 1 for Dereferenced variable may be null

@lgtm-com

lgtm-com Bot commented Feb 13, 2020

Copy link
Copy Markdown

This pull request introduces 1 alert when merging 09d6987 into 0912668 - view on LGTM.com

new alerts:

  • 1 for Dereferenced variable may be null

@sonarqubecloud

Copy link
Copy Markdown

SonarCloud Quality Gate failed.

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 3 Code Smells

26.2% 26.2% Coverage
9.6% 9.6% Duplication

@lgtm-com

lgtm-com Bot commented Feb 13, 2020

Copy link
Copy Markdown

This pull request introduces 1 alert when merging 3f215e9 into 0912668 - view on LGTM.com

new alerts:

  • 1 for Dereferenced variable may be null

@sonarqubecloud

Copy link
Copy Markdown

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 2 Security Hotspots to review)
Code Smell A 3 Code Smells

0.0% 0.0% Coverage
9.6% 9.6% Duplication

@lgtm-com

lgtm-com Bot commented Sep 10, 2020

Copy link
Copy Markdown

This pull request introduces 1 alert when merging 12e7e99 into 17aea8b - view on LGTM.com

new alerts:

  • 1 for Dereferenced variable may be null

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant