silverstripe / recipe-kitchen-sink

Silverstripe recipe combining all supported module for testing and releasing purposes.
BSD 3-Clause "New" or "Revised" License
2 stars 6 forks source link

Make sure all unsupported module have been removed form the CMS5 sink #29

Closed maxime-rainville closed 1 year ago

maxime-rainville commented 1 year ago

None of these modules should be on the list.

Module Status Notes
bringyourownideas/silverstripe-composer-security-checker Dropped Underlying service has been discontinued.
composer/installers Dropped Composer is more established than Silverstripe CMS.
cwp-themes/default Dropped
cwp/cwp CMS5 compatible without support
cwp/cwp-core CMS5 compatible without support
cwp/cwp-pdfexport Dropped
cwp/cwp-search CMS5 compatible without support
hafriedlander/phockito Dropped
hafriedlander/silverstripe-phockito Dropped
lekoala/silverstripe-debugbar CMS5 compatible without support Debug bar is a development tool that should not be installed in production.
silverstripe/activedirectory Dropped
silverstripe/akismet Dropped The parent library is outdated and they are better alternatives like UndefinedOffset/silverstripe-nocaptcha
silverstripe/behat-extension CMS5 compatible without official support
silverstripe/ckan-registry CMS5 compatible without support
silverstripe/comment-notifications CMS5 compatible without official support
silverstripe/comments CMS5 compatible without official support
silverstripe/content-widget CMS5 compatible without support
silverstripe/fulltextsearch CMS5 compatible without support
silverstripe/graphql-devtools CMS5 compatible without official support
silverstripe/html5 Folded back in core
silverstripe/postgresql CMS5 compatible without support
silverstripe/recipe-ccl CMS5 compatible without support
silverstripe/recipe-solr-search CMS5 compatible without support
silverstripe/security-extensions Folded back in core
silverstripe/spellcheck Dropped
silverstripe/sqlite3 CMS5 compatible without support
silverstripe/widgets CMS5 compatible without official support
tijsverkoyen/akismet Dropped
tractorcow/classproxy CMS5 compatible without official support
tractorcow/silverstripe-proxy-db CMS5 compatible without official support
undefinedoffset/sortablegridfield Dropped symbiote/silverstripe-gridfieldextensions provides equivalent functionality.

Acceptance criteria

PRs

GuySartorelli commented 1 year ago

For CMS 5 compatible versions, we should keep these in the sink until the RC at least, so we can be sure they actually are compatible.

emteknetnz commented 1 year ago

Some notes I took to work out what actually needs to be updated:

Supported modules reference

Not in 5 sink:

{x} bringyourownideas/silverstripe-composer-security-checker

{x} cwp-themes/default

{x} cwp/cwp-pdfexport

{x} hafriedlander/phockito

{x} hafriedlander/silverstripe-phockito

{x} lekoala/silverstripe-debugbar

{x} silverstripe/activedirectory

{x} silverstripe/akismet

{x} silverstripe/postgresql

{x} silverstripe/security-extensions

{x} silverstripe/spellcheck

{x} silverstripe/sqlite3

{x} tijsverkoyen/akismet

{x} undefinedoffset/sortablegridfield


require-dev or not in cow plan:

{-} tractorcow/silverstripe-proxy-db (as silverstripe/silverstripe-proxy-db)

{-} tractorcow/classproxy

{-} silverstirpe/behat-extension

{-} silverstripe/graphql-devtools

{-} composer/installers


Will be removed by removal of other modules:

{w} cwp/cwp-core

{w} cwp/cwp-search

{w} silverstripe/widgets

{w} silverstripe/fulltextsearch

===

A)

Remove from composer.json in silverstripe/recipe-kitchen-sink

{ } silverstripe/recipe-ccl

{ } silverstripe/recipe-solr-search

{ } silverstripe/ckan-registry

{ } cwp/cwp

===

B)

Remove from composer.json in silverstripe/recipe-blog:

{ } silverstripe/comment-notifications

{ } silverstripe/content-widget

{ } silverstripe/comments

===

C)

Ensure these are not in plan / upgrade-only / not in changelog

{-} cwp/cwp

====

maxime-rainville commented 1 year ago

All PRs have been merged.