status-im
/
status-mobile
mirror of https://github.com/status-im/status-mobile.git
2
0
Fork 0
Code Issues Projects Releases Wiki Activity

Raise clj-kondo linter levels (#17289) 

Recently, we changed clj-kondo default fail-level from "warning" to "error", but
we missed the fact that we needed to raise the default level for all linters set
to "warning".
This commit is contained in:
Icaro Motta 2023-09-14 11:28:43 -03:00 committed by GitHub
parent 07d1b13520
commit e2f837fcbc
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 90 additions and 62 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

59
.clj-kondo/config.edn
View File

@ -9,22 +9,49 @@
status-im.utils.styles/defn clojure.core/defn status-im.utils.styles/defn clojure.core/defn
test-helpers.unit/deftest-sub clojure.core/defn test-helpers.unit/deftest-sub clojure.core/defn
taoensso.tufte/defnp clojure.core/defn} taoensso.tufte/defnp clojure.core/defn}
:linters {:consistent-alias {:level :error :linters {:clj-kondo-config {:level :error}
:aliases {clojure.string string :cond-else {:level :error}
clojure.set set :consistent-alias {:level :error
clojure.walk walk :aliases {clojure.string string
taoensso.timbre log}} clojure.set set
:shadowed-var {:level :error clojure.walk walk
;; We temporarily use :include to define an taoensso.timbre log}}
;; allowlist of core Clojure vars. In the :docstring-blank {:level :error}
;; future, as we progressively fix shadowed :inline-def {:level :error}
;; vars, we should be able to delete this :invalid-arity {:skip-args [status-im.utils.fx/defn utils.re-frame/defn]}
;; option and lint all vars. :loop-without-recur {:level :error}
:exclude [type name]} :misplaced-docstring {:level :error}
:invalid-arity {:skip-args [status-im.utils.fx/defn utils.re-frame/defn]} :missing-body-in-when {:level :error}
:missing-clause-in-try {:level :error}
:missing-else-branch {:level :error}
:not-empty? {:level :error}
:quoted-case-test-constant {:level :error}
:redundant-do {:level :error}
:redundant-let {:level :error}
:refer-all {:level :error}
:shadowed-var {:level :error
;; We temporarily use :include to define an
;; allowlist of core Clojure vars. In the
;; future, as we progressively fix shadowed
;; vars, we should be able to delete this
;; option and lint all vars.
:exclude [type name]}
:single-operand-comparison {:level :error}
:syntax {:level :error}
:unbound-destructuring-default {:level :error}
:unknown-require-option {:level :error}
:unreachable-code {:level :error}
:unresolved-namespace {:level :error}
;; TODO remove number when this is fixed ;; TODO remove number when this is fixed
;; https://github.com/borkdude/clj-kondo/issues/867 ;; https://github.com/borkdude/clj-kondo/issues/867
:unresolved-symbol {:exclude [PersistentPriorityMap.EMPTY :unresolved-symbol {:exclude [PersistentPriorityMap.EMPTY
number number
status-im.test-helpers/restore-app-db]}} status-im.test-helpers/restore-app-db]}
:unresolved-var {:level :error}
:unused-binding {:level :error}
:unused-import {:level :error}
:unused-namespace {:level :error}
:unused-private-var {:level :error}
:unused-referred-var {:level :error}
:use {:level :error}}
:config-in-ns {mocks.js-dependencies {:linters {:clojure-lsp/unused-public-var {:level :off}}}}} :config-in-ns {mocks.js-dependencies {:linters {:clojure-lsp/unused-public-var {:level :off}}}}}

88
src/status_im/signals/core.cljs
View File

@ -96,52 +96,52 @@
^js event-js (.-event data) ^js event-js (.-event data)
type (.-type data)] type (.-type data)]
(case type (case type
"node.login" (profile.login/login-node-signal cofx (transforms/js->clj event-js)) "node.login" (profile.login/login-node-signal cofx (transforms/js->clj event-js))
"backup.performed" {:db (assoc-in db "backup.performed" {:db (assoc-in db
[:profile/profile :last-backup] [:profile/profile :last-backup]
(.-lastBackup event-js))} (.-lastBackup event-js))}
"envelope.sent" (transport.message/update-envelopes-status cofx "envelope.sent" (transport.message/update-envelopes-status cofx
(:ids (:ids
(js->clj event-js
:keywordize-keys
true))
:sent)
"envelope.expired" (transport.message/update-envelopes-status cofx
(:ids
(js->clj event-js
:keywordize-keys
true))
:not-sent)
"message.delivered" (let [{:keys [chatID messageID]} (js->clj event-js
:keywordize-keys
true)]
(models.message/update-db-message-status cofx
chatID
messageID
:delivered))
"mailserver.changed" (mailserver/handle-mailserver-changed cofx (.-id event-js))
"mailserver.available" (mailserver/handle-mailserver-available cofx (.-id event-js))
"mailserver.not.working" (mailserver/handle-mailserver-not-working cofx)
"discovery.summary" (summary cofx (js->clj event-js :keywordize-keys true))
"mediaserver.started" {:db (assoc db :mediaserver/port (.-port event-js))}
"wakuv2.peerstats" (wakuv2-peer-stats cofx (js->clj event-js :keywordize-keys true))
"messages.new" (transport.message/sanitize-messages-and-process-response cofx
event-js
true)
"wallet" (ethereum.subscriptions/new-wallet-event cofx
(js->clj event-js (js->clj event-js
:keywordize-keys :keywordize-keys
true)) true))
"local-notifications" (local-notifications/process cofx :sent)
(js->clj event-js :keywordize-keys true)) "envelope.expired" (transport.message/update-envelopes-status cofx
"community.found" (link-preview/cache-community-preview-data (js->clj event-js (:ids
:keywordize-keys (js->clj event-js
true)) :keywordize-keys
"status.updates.timedout" (visibility-status-updates/handle-visibility-status-updates true))
cofx :not-sent)
(js->clj event-js :keywordize-keys true)) "message.delivered" (let [{:keys [chatID messageID]} (js->clj event-js
"localPairing" (handle-local-pairing-signals :keywordize-keys
cofx true)]
(js->clj event-js :keywordize-keys true)) (models.message/update-db-message-status cofx
chatID
messageID
:delivered))
"mailserver.changed" (mailserver/handle-mailserver-changed cofx (.-id event-js))
"mailserver.available" (mailserver/handle-mailserver-available cofx (.-id event-js))
"mailserver.not.working" (mailserver/handle-mailserver-not-working cofx)
"discovery.summary" (summary cofx (js->clj event-js :keywordize-keys true))
"mediaserver.started" {:db (assoc db :mediaserver/port (.-port event-js))}
"wakuv2.peerstats" (wakuv2-peer-stats cofx (js->clj event-js :keywordize-keys true))
"messages.new" (transport.message/sanitize-messages-and-process-response cofx
event-js
true)
"wallet" (ethereum.subscriptions/new-wallet-event cofx
(js->clj event-js
:keywordize-keys
true))
"local-notifications" (local-notifications/process cofx
(js->clj event-js :keywordize-keys true))
"community.found" (link-preview/cache-community-preview-data (js->clj event-js
:keywordize-keys
true))
"status.updates.timedout" (visibility-status-updates/handle-visibility-status-updates
cofx
(js->clj event-js :keywordize-keys true))
"localPairing" (handle-local-pairing-signals
cofx
(js->clj event-js :keywordize-keys true))
(log/debug "Event " type " not handled")))) (log/debug "Event " type " not handled"))))

5
src/status_im2/contexts/chat/messages/content/text/view.cljs
View File

@ -6,6 +6,7 @@
[status-im2.contexts.chat.messages.content.link-preview.view :as link-preview] [status-im2.contexts.chat.messages.content.link-preview.view :as link-preview]
[status-im2.contexts.chat.messages.content.text.style :as style] [status-im2.contexts.chat.messages.content.text.style :as style]
[utils.i18n :as i18n] [utils.i18n :as i18n]
[react-native.platform :as platform]
[utils.re-frame :as rf])) [utils.re-frame :as rf]))
(defn render-inline (defn render-inline
@ -104,8 +105,8 @@
(render-inline acc e chat-id style-override mention-first)) (render-inline acc e chat-id style-override mention-first))
[quo/text [quo/text
{:style {:size :paragraph-1 {:style {:size :paragraph-1
:margin-bottom (if mention-first (if quo.platform/ios? 4 0) 2) :margin-bottom (if mention-first (if platform/ios? 4 0) 2)
:margin-top (if mention-first (if quo.platform/ios? -4 0) 2) :margin-top (if mention-first (if platform/ios? -4 0) 2)
:color (when (seq style-override) colors/white)}}] :color (when (seq style-override) colors/white)}}]
children)]) children)])