Skip to content

Update Sonarcloud Properties#1354

Merged
olabusayoT merged 1 commit intoapache:mainfrom
olabusayoT:daf-2949-sonarCloudPropertiesUpdates
Oct 28, 2024
Merged

Update Sonarcloud Properties#1354
olabusayoT merged 1 commit intoapache:mainfrom
olabusayoT:daf-2949-sonarCloudPropertiesUpdates

Conversation

@olabusayoT
Copy link
Copy Markdown
Contributor

  • currently we check Main.scala for duplicate code, which is a problem due to DAFFODIL-1958. So we update the path to Main.scala so we can continue to exclude it.
  • we no longer have src/it in our codebase, so we remove that from tests
  • Added in 3 newest modules since their absence seems to be an oversight

DAFFODIL-2949

- currently we check Main.scala for duplicate code, which is a problem due to DAFFODIL-1958. So we update the path to Main.scala so we can continue to exclude it.
- we no longer have src/it in our codebase, so we remove that from tests
- Added in 3 newest modules since their absence seems to be an oversight

DAFFODIL-2949
Copy link
Copy Markdown
Member

@stevedlawrence stevedlawrence left a comment

Choose a reason for hiding this comment

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

+1

Copy link
Copy Markdown
Contributor

@jadams-tresys jadams-tresys left a comment

Choose a reason for hiding this comment

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

+1

@olabusayoT olabusayoT merged commit c802eb8 into apache:main Oct 28, 2024
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.

3 participants