Skip to content

update scala-xml#237

Merged
farmdawgnation merged 1 commit into
dispatch:mainfrom
yanns:235_update_scala_xml
Feb 2, 2023
Merged

update scala-xml#237
farmdawgnation merged 1 commit into
dispatch:mainfrom
yanns:235_update_scala_xml

Conversation

@yanns

@yanns yanns commented Jan 2, 2023

Copy link
Copy Markdown
Contributor

fix #235
lift-json also depends on scala-xml 1.2.
lift-json is not maintained anymore.
I propose to remove the module.

fix dispatch#235
lift-json also depends on scala-xml 1.2.
lift-json is not maintained anymore.
I propose to remove the module.
@yanns

yanns commented Jan 30, 2023

Copy link
Copy Markdown
Contributor Author

@farmdawgnation can you authorize the PR checks?

@farmdawgnation

farmdawgnation commented Jan 30, 2023 via email

Copy link
Copy Markdown
Member

@farmdawgnation

Copy link
Copy Markdown
Member

PR checks are authorized. Will noodle on how I feel about actually pulling the plug on lift-json support but alas, I'm also one of the key maintainers over there and I'm not sure it's aged particularly well.

@farmdawgnation

Copy link
Copy Markdown
Member

Checks have all passed... give me a bit to think about the lift-json question.

@farmdawgnation farmdawgnation left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Approved, though I think I'll have to consider this a major version bump.

@farmdawgnation farmdawgnation merged commit 658a1ba into dispatch:main Feb 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

Update scala-xml dependency to version 2

2 participants