Skip to content
This repository was archived by the owner on Oct 17, 2022. It is now read-only.

Remove mentions on external process shenanigans - #276

Merged
janl merged 2 commits into
masterfrom
update-notifications-lol
Jul 13, 2018
Merged

Remove mentions on external process shenanigans#276
janl merged 2 commits into
masterfrom
update-notifications-lol

Conversation

@janl

@janl janl commented May 16, 2018

Copy link
Copy Markdown
Member

This never really made it to 2.x and won’t be a thing going forward. At the moment it only confuses folks by being documented.

@janl
janl requested review from garbados and wohali May 16, 2018 16:15
@janl
janl force-pushed the update-notifications-lol branch from f5f285f to c04f34d Compare May 16, 2018 16:26
@wohali

wohali commented May 16, 2018

Copy link
Copy Markdown
Member

We need to remove all of this from couch as well:

default.ini:os_daemons={couch_os_daemons, start_link, []}
default.ini:; [os_daemons]
default.ini:; [os_daemon_settings]
local.ini:[os_daemons]
local.ini:[update_notification]

@wohali wohali 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.

We shouldn't remove all of this until the matching code is gone from couchdb, or at least the docs in *.ini.

@janl

janl commented May 16, 2018

Copy link
Copy Markdown
Member Author

@wohali done in apache/couchdb#1330

@wohali

wohali commented Jul 12, 2018

Copy link
Copy Markdown
Member

You have a +1, merge away

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants