Skip to content

feat(yesod): Use data-default 0.8 - #1936

Closed
Vekhir wants to merge 1 commit into
yesodweb:masterfrom
Vekhir:data-default-0.8
Closed

Vekhir wants to merge 1 commit into
yesodweb:masterfrom
Vekhir:data-default-0.8

Conversation

@Vekhir

@Vekhir Vekhir commented Oct 9, 2026 •

Copy link
Copy Markdown
Contributor

data-default 0.8.0.0 incorporates data-default-class and moves the Default (def) constructor into a different namespace.
data-default-class 0.2.0.0 re-exports the module for compatibility.

Introduce the new flag "data-default", with which yesod uses data-default directly. To keep the CI passing, it is off by default; at some point in the future, it may be enabled by default.

Before submitting your PR, check that you've:

  • Bumped the version number

After submitting your PR:

  • Update the Changelog.md file with a link to your PR
  • Check that CI passes (or if it fails, for reasons unrelated to your change, like CI timeouts)

Vekhir added a commit to Vekhir/yesod that referenced this pull request Oct 9, 2026
Vekhir added a commit to Vekhir/yesod that referenced this pull request Oct 9, 2026
Vekhir added a commit to Vekhir/yesod that referenced this pull request Oct 9, 2026
Vekhir added a commit to Vekhir/yesod that referenced this pull request Oct 9, 2026
Vekhir added a commit to Vekhir/yesod that referenced this pull request Oct 9, 2026
Vekhir added a commit to Vekhir/yesod that referenced this pull request Oct 9, 2026
Vekhir added a commit to Vekhir/yesod that referenced this pull request Oct 9, 2026
@Vekhir

Vekhir commented Oct 15, 2026

Copy link
Copy Markdown
Contributor Author

Better to use data-default-class 0.2.0.0 which isn't restricted by the bounds.

@Vekhir Vekhir closed this Oct 15, 2026
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.

1 participant