Skip to content

Update several dependencies and plugins #525

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.

Sign up for GitHub

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

Jump to bottom
Merged
mofterdinger merged 11 commits into from
Aug 7, 2025
Merged

Update several dependencies and plugins #525

mofterdinger merged 11 commits into from
Aug 7, 2025

Conversation

@mofterdinger
Copy link
Member

@mofterdinger mofterdinger commented Aug 6, 2025 •
edited
Loading

Update the following components:

  • SAP UI5 to 1.139.0
  • updated package-lock.json in root and app folder to latest available versions
  • several Maven build plugins to latest available versions

@mofterdinger
Bump SAP UI5 to 1.138.1
e388015
@mofterdinger
Bump cds-services to 4.2.0
588b294
@mofterdinger
update package-lock.json to latest versions
f0eaa8d
@mofterdinger
updated approuter package-lock.json
eaf75e5
@mofterdinger
update package-lock.json
6151364
@mofterdinger
update build plugins:
aa72d65 
maven-compiler-plugin ............................ 3.13.0 -> 3.14.0
maven-enforcer-plugin .............................. 3.5.0 -> 3.6.1
maven-surefire-plugin .............................. 3.5.2 -> 3.5.3
org.codehaus.mojo:flatten-maven-plugin ............. 1.6.0 -> 1.7.2
@mofterdinger mofterdinger changed the title Update several dependencies Update several dependencies and plugins Aug 6, 2025
agoerler
agoerler reviewed Aug 7, 2025
View reviewed changes
Copy link
Contributor

@agoerler agoerler left a comment

Choose a reason for hiding this comment

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

Do we also need to adjust this:

https://github.wdf.sap.corp/cds-java/cds-services/commit/d26d912c6b76a6bd12aeeffb774f4e40e2df00b9

?

@MattSchur
Copy link
Contributor

MattSchur commented Aug 7, 2025

We also have author.name in the header, this seems to cause the failing PATCH on Books(ID)/author

@mofterdinger
Copy link
Member Author

mofterdinger commented Aug 7, 2025

Hi, I added "editableHeaderContent": false to manifest.json, but without success. I'm still seeing getting the same error.

@mofterdinger
Copy link
Member Author

mofterdinger commented Aug 7, 2025 •
edited
Loading

After fixing another CDS syntax error in file app/common.cds it works as expected: dd5d451

@mofterdinger mofterdinger requested a review from agoerler August 7, 2025 12:59
@mofterdinger mofterdinger merged commit d625f80 into postgres Aug 7, 2025
2 checks passed
@mofterdinger mofterdinger deleted the pg_version_bump branch August 7, 2025 14:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@agoerler agoerler agoerler approved these changes

@MattSchur MattSchur MattSchur approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@mofterdinger @MattSchur @agoerler