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

Veracode Fixes #24

Merged
merged 22 commits into from
May 8, 2023
Merged

Veracode Fixes #24

merged 22 commits into from
May 8, 2023

Conversation

adkumar1
Copy link
Contributor

@adkumar1 adkumar1 commented Apr 21, 2023

First merge PR #23

Copy link
Contributor

@sachinargade123 sachinargade123 left a comment

Choose a reason for hiding this comment

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

This changes looks good

@sachinargade123
Copy link
Contributor

This PR should be merge after PR #23

@adkumar1 adkumar1 mentioned this pull request May 5, 2023
<dependency>
<groupId>org.springframework</groupId>
<artifactId>spring-webmvc</artifactId>
<version>6.0.8</version>
Copy link
Contributor

Choose a reason for hiding this comment

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

How is it possible that you update spring-webmvc but the DEPENCY File is not reflecting this change?

Have you not updated/run DASH again? :(

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@sachinargade123 : Please check

Copy link
Contributor

Choose a reason for hiding this comment

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

Yes, DASH tool was not run for this commit, DEPENCY file update for next PR #31.

@Siegfriedk Siegfriedk merged commit c8b019a into eclipse-tractusx:main May 8, 2023
@Siegfriedk Siegfriedk deleted the R1.2.7 branch May 8, 2023 07:20
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