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.

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

Update some dependencies #1910

Merged
merged 4 commits into from
Dec 21, 2023
Merged

Update some dependencies #1910

merged 4 commits into from
Dec 21, 2023

Conversation

ymo-sci
Copy link
Contributor

@ymo-sci ymo-sci commented Dec 14, 2023

Description

see commits

Additional Notes

Checklist

  • Code change has been tested and works locally
  • Code was formatted via IntelliJ and follows SonarLint & best practices
  • Patch Tasks: Is local execution of Patch Tasks necessary? If so, please also mark the PR with the tag.

@ymo-sci
update some dependencies.
60850a8 
commons-io is already present in the newest version in sirius-kernel and not needed here.
snakeyml is only needed in biz, so we moved it out of kernel to biz

- fixes: SIRI-920
@ymo-sci
update deprecated methods to newer ones
b4b3098 
- fixes: SIRI-920
@ymo-sci ymo-sci added 🖐 Keep open Should not be merged 🕔 Wait for sirius Needs new version from another sirius lib to work ⬆️ Dependencies Pull requests that update a dependency file labels Dec 14, 2023
@ymo-sci ymo-sci changed the title Feature/ymo/siri 920 dependencies Update some dependencies Dec 14, 2023
@ymo-sci ymo-sci removed 🖐 Keep open Should not be merged 🕔 Wait for sirius Needs new version from another sirius lib to work labels Dec 19, 2023
@ymo-sci ymo-sci merged commit 1c0f210 into develop Dec 21, 2023
2 checks passed
@ymo-sci ymo-sci deleted the feature/ymo/SIRI-920-dependencies branch December 21, 2023 11:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bkositza bkositza bkositza approved these changes

@mkeckmkeck mkeckmkeck mkeckmkeck approved these changes

@idlira idlira idlira approved these changes

@jakobvogel jakobvogel jakobvogel approved these changes

@jmuscireum jmuscireum jmuscireum approved these changes

Assignees
No one assigned
Labels
⬆️ Dependencies Pull requests that update a dependency file
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@ymo-sci @jmuscireum @jakobvogel @idlira @mkeckmkeck @bkositza