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.

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

build: add header check to comply with TRG 7.02 #1029

Merged
merged 4 commits into from
Feb 7, 2024

Conversation

ndr-brt
Copy link
Contributor

@ndr-brt ndr-brt commented Feb 7, 2024

WHAT

Add a check on checkstyle to ensure the headers are in the correct format according to TRG 7.02

WHY

TRG compliance

FURTHER NOTES

List other areas of code that have changed but are not necessarily linked to the main feature. This could be method signature changes, package declarations, bugs that were encountered and were fixed inline, etc.

Closes #88

@ndr-brt ndr-brt added the enhancement New feature or request label Feb 7, 2024
@ndr-brt ndr-brt force-pushed the 88-headers-check branch 5 times, most recently from ff4ea26 to a8212e2 Compare February 7, 2024 09:28
@ndr-brt ndr-brt marked this pull request as ready for review February 7, 2024 12:31
Copy link
Contributor

@wolf4ood wolf4ood left a comment

Choose a reason for hiding this comment

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

🚀

build.gradle.kts Show resolved Hide resolved
@ndr-brt ndr-brt force-pushed the 88-headers-check branch 4 times, most recently from 9578941 to 0750ce7 Compare February 7, 2024 14:26
Copy link

sonarqubecloud bot commented Feb 7, 2024

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@ndr-brt ndr-brt merged commit 5dda6b9 into eclipse-tractusx:main Feb 7, 2024
26 checks passed
@ndr-brt ndr-brt deleted the 88-headers-check branch February 7, 2024 15:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Merged
Development

Successfully merging this pull request may close these issues.

Fix license header to match Eclipse recommendations
3 participants