Remove builtIn dependency values #268

Merged
luhahn merged 1 commits from refs/pull/268/head into master 2021-12-22 15:41:36 +00:00
luhahn commented 2021-12-21 11:24:56 +00:00 (Migrated from gitea.com)

⚠️ Breaking

Moved the values to enable the dependencies into the dependencies itself, this way we don't need a seperate field in the values and it is more obvious how to enable for example postgresql.

⚠️ Breaking Moved the values to enable the dependencies into the dependencies itself, this way we don't need a seperate field in the values and it is more obvious how to enable for example postgresql.
justusbunsi (Migrated from gitea.com) reviewed 2021-12-21 12:06:05 +00:00
justusbunsi (Migrated from gitea.com) left a comment

README needs to be modified. The other parts LGTM.

README needs to be modified. The other parts LGTM.
justusbunsi (Migrated from gitea.com) reviewed 2021-12-21 12:23:01 +00:00
justusbunsi (Migrated from gitea.com) reviewed 2021-12-21 13:15:08 +00:00
justusbunsi (Migrated from gitea.com) approved these changes 2021-12-21 13:26:40 +00:00
justusbunsi (Migrated from gitea.com) left a comment

If I didn't miss anything, LGTM now.

If I didn't miss anything, LGTM now.
justusbunsi commented 2021-12-22 12:30:08 +00:00 (Migrated from gitea.com)

My approval is still valid after reviewing the latest changes. ?

My approval is still valid after reviewing the latest changes. ?
wxiaoguang (Migrated from gitea.com) reviewed 2021-12-22 13:20:17 +00:00
zeripath (Migrated from gitea.com) approved these changes 2021-12-22 14:39:32 +00:00
Sign in to join this conversation.
No description provided.