Skip to content

Conversation

@f-f
Copy link
Collaborator

@f-f f-f commented Oct 29, 2025

Description

Fix #5371

Checklist

  • Commits in meaningful sequence and with useful messages.
  • Tests added or updated when needed.
  • CHANGELOG.md files updated for packages with externally visible changes.
    NOTE: New section is never added with the code changes. (See RELEASING.md).
  • Versions updated in .cabal and CHANGELOG.md files when necessary, according to the
    versioning process.
  • Version bounds in .cabal files updated when necessary.
    NOTE: If bounds change in a cabal file, that package itself must have a version increase. (See RELEASING.md).
  • Code formatted (use scripts/fourmolize.sh).
  • Cabal files formatted (use scripts/cabal-format.sh).
  • CDDL files are up to date (use scripts/gen-cddl.sh)
  • hie.yaml updated (use scripts/gen-hie.sh).
  • Self-reviewed the diff.

@f-f f-f requested a review from a team as a code owner October 29, 2025 18:09
@f-f
Copy link
Collaborator Author

f-f commented Oct 29, 2025

Deleting code is great 😄

I assume the changelog doesn't need adjusting since this was all previously marked as changed/removed?

Copy link
Collaborator

@lehins lehins left a comment

Choose a reason for hiding this comment

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

This is an awesome cleanup!
Unfortunately, we do need to mention all of the things that were removed by this PR in changelogs 😢
Moreover, any package that had anything removed needs a major version bump, if it didn't already have one, because any removal is considered a breaking change according to PVP.

@f-f f-f force-pushed the f-f/fix-5371 branch 2 times, most recently from 9e75aed to 5d20c68 Compare October 31, 2025 10:07
@f-f
Copy link
Collaborator Author

f-f commented Oct 31, 2025

@lehins I have updated all the relevant changelogs so this should be ready for another look

Copy link
Collaborator

@lehins lehins left a comment

Choose a reason for hiding this comment

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

This is perfect! Thank you!

@lehins
Copy link
Collaborator

lehins commented Oct 31, 2025

Needs a rebase on master, but other than that it is ready to be merged.

@f-f f-f merged commit 173fc6b into master Nov 3, 2025
121 of 123 checks passed
@f-f f-f deleted the f-f/fix-5371 branch November 3, 2025 13:48
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.

Remove all deprecated functionality.

3 participants