Authentication-Results: mail-b.sr.ht; dkim=fail header.d=thorlaksson.com header.i=@thorlaksson.com Received: from out2.migadu.com (out2.migadu.com [188.165.223.204]) by mail-b.sr.ht (Postfix) with ESMTPS id 044DC11EEF6 for <~reykjalin/public-inbox@lists.sr.ht>; Wed, 21 Apr 2021 19:27:53 +0000 (UTC) MIME-Version: 1.0 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=thorlaksson.com; s=key1; t=1619033272; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=u+rlf/aAn0u3X3/cf01z8vEXpI5ThZJK+idfBBYs1VA=; b=BrkcmILNAH8WV9YewVXAo/Lb0tWouO6/zWMQtGuSypfn6xKPcISCFjMc/6O2Qu0ALaS0f/ 5mfLqeGoTOy1hHiS8k6ho0fTuxfgGLM9HsMNRjO+yYK/LBeVhGGKNEknASSKZhQs6UAiEq 0HUFhtDOfmFOJD2UwAWdB8cwmKxAtOAA0hCTL2kza4xskfxOjrBxZYyyTqR/RlpO2Fe2Dv RxYiGGyn/eiU/3AAj3NeKahvTBwa9i9GFmb6QPKV16vemMbKNgQJVO7DQ3LI0Z9Ylst7wt vFfM24eICpvAe6PZ1Nxdh9sYMrewm2zuQb+llMX4UOVsfzLr/8HMgDR8/Yieuw== Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset=UTF-8 Date: Wed, 21 Apr 2021 19:27:50 +0000 Message-Id: Cc: Subject: Re: Possible bug in nova-phpcs X-Report-Abuse: Please report any abuse attempt to abuse@migadu.com and include these headers. From: =?utf-8?q?Krist=C3=B3fer_Reykjal=C3=ADn?= To: "Guilherme Cartaxo" , <~reykjalin/public-inbox@lists.sr.ht> References: <949DA8B7-8F82-47D2-BE1B-A58813656659@weareboq.com> In-Reply-To: <949DA8B7-8F82-47D2-BE1B-A58813656659@weareboq.com> X-Migadu-Flow: FLOW_OUT X-Migadu-Auth-User: kristofer@thorlaksson.com Hi Guilherme, > After the last update, nova-phpcs started automatic fixing code on > every save. > > Today I finally decided to investigate and found that config.js seems > to be loading phpcbf instead of phpcs in lines 16 and 20. Thank you so much for reporting this! I don't know how I haven't seen this since I've been using the extension like this since the last release :O . I'm so sorry about the issue! I hope it didn't cause any issues in your workflows. For now, you can quickly monkey patch this yourself by manually modifying config.js in "~/Library/Application Support/Nova/Extensions/com.thorlaksson.phpcs/Script= s/config.js" (at least I think that's the right path). Unfortunately I don't have the time at the moment to release a fix, but you can expect a release with an update this coming Monday (April 26). Thank you again for reporting this! I'll be in touch again once I've fixed the issue, just to make sure everything works as intended :) . Best regards, Krist=C3=B3fer R.