Teamwork/node-auto-launch

Add `toggle` method for easy enable/disable

Open

#21 opened on Feb 10, 2016

View on GitHub
 (1 comment) (1 reaction) (1 assignee)CoffeeScript (911 stars) (111 forks)batch import
enhancementgood first issuehelp wanted

Description

Add toggle method for easy enable/disable. This makes it easy to set based based on a variable.

Current:

var somebool;
autoLauncher[(somebool ? 'enable' : 'disable')](function(err) {
   console.log(err);
});
// or
(somebool ? autoLauncher.enable : autoLauncher.disable).bind(autoLauncher)(function(err){
   console.log(err);
});

Desired:

var somebool;
autoLauncher.toggle(somebool, function(err) {
   console.log(err);
});

Contributor guide

Tech stack
javascriptnodejs
Domain
desktop
Issue type
feature
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
under 1 hour
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
blocked
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
clear
Prerequisites
None
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
50
Research direction
Review the existing enable and disable methods in the source code (likely in CoffeeScript). The toggle method should accept a boolean and callback, then delegate to enable/disable. No additional context or linked PRs.