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

Fix for TRG checks of QG 24.05 #184

Merged
merged 6 commits into from
May 15, 2024
Merged

Fix for TRG checks of QG 24.05 #184

merged 6 commits into from
May 15, 2024

Conversation

fabiodmota
Copy link
Contributor

Description

Fix for TRG checks of QG 24.05

fixes:

Pre-review checks

Please ensure to do as many of the following checks as possible, before asking for committer review:

Copy link

@evegufy evegufy left a comment

Choose a reason for hiding this comment

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

Looks good in general, just one comment

frontend/LICENSES/CC-BY-4.0.txt Outdated Show resolved Hide resolved
Copy link

@evegufy evegufy left a comment

Choose a reason for hiding this comment

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

looks good, but forgot to add that yesterday in the helm improvement issue: could you please also still remove the helper dir https://github.com/eclipse-tractusx/vas-country-risk/tree/fix/QG-TRGs/charts/helper, you mentioned it is not used and I find it confusing to have there then

@fabiodmota fabiodmota requested a review from evegufy May 15, 2024 14:22
@fabiodmota fabiodmota merged commit ac4347d into main May 15, 2024
10 checks passed
@fabiodmota fabiodmota deleted the fix/QG-TRGs branch May 15, 2024 14:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

2 participants