Skip to content
This repository has been archived by the owner on Jun 8, 2023. It is now read-only.

Removed drupalorg.coffee script #1175

Closed
wants to merge 2 commits into from
Closed

Removed drupalorg.coffee script #1175

wants to merge 2 commits into from

Conversation

patcon
Copy link
Contributor

@patcon patcon commented Oct 8, 2013

This script now exists in an external package repo.
https://github.com/myplanetdigital/hubot-drupalorg

Not sure how you guys like to handle this.

cc: @guyoron

@guyoron
Copy link
Contributor

guyoron commented Oct 8, 2013

Hey @patcon, thanks for cc'ing me. I'm curious, why move it from this repo to its own repo? Having it here means it's easier to discover, and that it can be used in the same way all hubot scripts are used.

@technicalpickles
Copy link
Contributor

@patcon instead of removing, maybe a deprecation is a better first step, and give a warning it will be removed in the next major release?

@guyoron hubot-scripts is actually really, really difficult to maintain. Check out #1113 for some background

@guyoron
Copy link
Contributor

guyoron commented Oct 8, 2013

ah thanks. Didn't realize #1113 was happening.

@patcon
Copy link
Contributor Author

patcon commented Oct 25, 2013

Where does a deprecation warning best happen? Any other scripts doing things I can copy?

@technicalpickles
Copy link
Contributor

Not yet. You can just call robot.logger.warning inside the modules.exports = (robot) -> call.

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