Lint

.erb-lint.yml

@@ -0,0 +1,30 @@
+---
+
+linters:
+  ExtraNewline:
+    enabled: true
+
+  FinalNewline:
+    enabled: true
+
+  SpaceAroundErbTag:
+    enabled: true
+
+  AllowedScriptType:
+    enabled: true
+    allowed_types:
+      - text/javascript
+      - text/template
+
+  Rubocop:
+    enabled: true
+
+    rubocop_config:
+      AllCops:
+        DisabledByDefault: true
+
+      Style/StringLiterals:
+        EnforcedStyle: double_quotes
+
+      Layout/SpaceInsideHashLiteralBraces:
+        Enabled: true

.eslintignore

@@ -0,0 +1,15 @@
+**/*{.,-}min.js
+decidim-*/vendor/**/*.js
+spec/decidim_dummy_app/**/*.js
+development_app
+**/node_modules/**
+bundle.js
+karma.conf.js
+webpack.config.js
+webpack.config.babel.js
+entry.test.js
+entry.js
+*_manifest.js
+coverage
+decidim-dev/**/*/test/**/*.js
+vendor/bundle

.eslintrc.json

@@ -0,0 +1,3 @@
+{
+  "extends": "@decidim"
+}

.github/workflows/lint.yml

@@ -28,13 +28,20 @@ jobs:
         with:
           node-version: ${{ env.NODE_VERSION }}
 
+      - run: npm ci
+        name: Install JS deps
+
       - run: bundle exec rubocop -P
         name: Lint Ruby files
 
       - run: bundle exec mdl *.md
         name: Lint Markdown files
 
-      - name: Lint and Rubocop
-        run: |
-          bundle exec rubocop -P
-          bundle exec erblint app/**/*.erb
+      - run: bundle exec erblint {decidim-module-catcomu_managers/,}app/{cells,views}/**/*.erb
+        name: Lint ERB files
+
+      - run: npm run stylelint
+        name: Lint SCSS files
+
+      - run: npm run lint
+        name: Lint JS files

app/packs/src/decidim/decidim_application.js

@@ -3,4 +3,6 @@
 
 // Load images
 require.context("../../images", true)
+
+// Load fonts
 require.context("../../fonts", true)

app/packs/stylesheets/catencomu/partials/_body.scss

@@ -31,7 +31,7 @@ h4,
 h5,
 .h5,
 h6,
-.h6{
+.h6 {
   font-family: 'Montserrat Bold', sans-serif;
   font-style: normal;
   font-weight: normal;

app/packs/stylesheets/catencomu/partials/_cards.scss

@@ -6,4 +6,4 @@
   &:hover {
     color: $card-link-hover;
   }
-}
+}

app/packs/stylesheets/catencomu/partials/_footer.scss

@@ -3,18 +3,19 @@
 
 .main-footer {
   @include gradient-background($catencomu-red, $catencomu-lilac);
-  
+
   a {
     color: $link-color;
   }
+
   .main-footer__badge {
     position: unset;
   }
 }
 
-.mini-footer{
+.mini-footer {
   @include gradient-background($catencomu-red, $catencomu-lilac);
-  
+
   a {
     color: $link-color;
   }

app/packs/stylesheets/catencomu/partials/_links.scss

@@ -4,47 +4,60 @@
 .cookie-warning a {
   color: var(--secondary);
 }
+
 .button.hollow.secondary:hover,
 .button.hollow.secondary:focus {
   color: var(--secondary);
 }
+
 .floating-helper__text {
   color: var(--secondary);
 }
+
 .floating-helper__icon {
   background-color: var(--secondary);
 }
+
 .global-accountability .row.column .card--accountability .card__content ul a,
 .global-accountability .row.columns .card--accountability .card__content ul a,
 .global-accountability .row.column .card--accountability .badge-card__content ul a,
 .global-accountability .row.columns .card--accountability .badge-card__content ul a {
   color: var(--secondary);
 }
+
 .topbar__dropmenu .is-dropdown-submenu li a {
   color: var(--secondary);
 }
+
 .svg-icon-path {
-  color: var(--secondary) !important ;
+  color: var(--secondary) !important;
 }
+
 html a {
   color: var(--secondary);
 }
+
 .card__link,
 .card__link:hover {
   color: var(--secondary);
 }
+
 .card__text a:hover {
   color: $body-font-color;
 }
+
 .author-data .author__name {
   color: var(--secondary);
 }
+
 .process_stats .process_stats-item .icon {
   color: var(--secondary);
 }
+
 #profile-tabs.tabs .tabs-title a {
   color: var(--secondary);
 }
+
 .floating-helper__icon-big {
   color: var(--secondary);
 }
@@ -57,38 +70,49 @@ html a {
     color: $body-font-color;
   }
 }
+
 .side-panel__tabs .tabs-title.is-active > [role="tab"],
 .side-panel__tabs .tabs-title.is-active > a {
   color: var(--secondary);
 }
+
 .side-panel__tabs .tabs-title > [role="tab"]:hover,
 .side-panel__tabs .tabs-title > a:hover {
   color: var(--secondary);
 }
+
 .opinion-toggle--meh:hover {
   background-color: var(--secondary);
 }
+
 .opinion-toggle--meh:focus {
   background-color: var(--secondary);
 }
+
 .opinion-toggle--meh.is-active {
   background-color: var(--secondary);
 }
+
 .floating-helper__content-close {
   background-color: var(--secondary);
 }
+
 .button.secondary {
   background-color: var(--secondary);
 }
+
 .floating-helper__content {
   border-top: 5px solid var(--secondary);
 }
+
 .pagination .current {
   background: var(--secondary);
 }
+
 .dropdown.menu > li.is-dropdown-submenu-parent > a::after {
   border-color: var(--secondary) transparent transparent;
 }
+
 .dropdown.menu .is-active > a {
   color: var(--secondary);
 }
@@ -101,21 +125,27 @@ html a {
 a:focus {
   color: var(--secondary);
 }
+
 a:hover {
   color: var(--secondary);
 }
+
 .author__name:hover {
   color: var(--secondary);
 }
+
 .author-data a:hover {
   color: var(--secondary);
 }
+
 .author-data button:hover {
   color: var(--secondary);
 }
+
 .button.secondary:hover {
   background-color: var(--secondary);
 }
+
 .card--list__icon {
   fill: var(--secondary);
 }

app/packs/stylesheets/catencomu/partials/_login.scss

@@ -1,7 +1,8 @@
 .button--civicrm {
   .button--social__icon {
     margin: 0;
-    padding: 0.3em 0;
+    padding: .3em 0;
+
     .icon {
       display: inline-block !important;
       border-radius: 4px;
@@ -13,8 +14,8 @@
   }
 }
 
-.social-register{
-  .button--social{
+.social-register {
+  .button--social {
     text-align: center;
   }
-}
+}

app/packs/stylesheets/catencomu/partials/_navbar.scss

@@ -4,16 +4,18 @@
 
 .title-bar {
   @include gradient-background($catencomu-red, $catencomu-lilac);
-  padding: 1.5rem 0.5rem;
+
+  padding: 1.5rem .5rem;
 }
 
 .navbar {
   @include gradient-background($catencomu-red, $catencomu-lilac);
+
   font-family: 'Montserrat Bold', sans-serif;
 
   .main-nav__link {
     a {
-      color: transparentize($navbar-bg, 0.3);
+      color: transparentize($navbar-bg, .3);
     }
 
     &--active a {
@@ -22,21 +24,20 @@
   }
 }
 
-
-.topbar_{
-  &_search{
+.topbar_ {
+  &_search {
     input,
-    input:focus{
+    input:focus {
       background-color: $navbar-bg;
       color: $navbar-fg-hover;
     }
 
     input::placeholder,
-    textarea::placeholder{
+    textarea::placeholder {
       color: $navbar-fg-hover;
     }
 
-    svg{
+    svg {
       color: $navbar-fg-hover;
     }
   }

app/packs/stylesheets/catencomu/partials/_variables.scss

@@ -21,7 +21,7 @@ $success: #6eeb83;
 $success-rgb:110,235,131;
 $warning: #f46036;
 $warning-rgb: 244,96,54;
-$alert: #EE3138;
+$alert: #ee3138;
 $alert-rgb: 239,45,86;
 
 $title-bar-bg: #080707;
@@ -45,7 +45,7 @@ $google: #dd4b39;
 $color-addition: #e6ffed;
 $color-removal: #ffeef0;
 
-:root{
+:root {
   --primary: #{$primary};
   --primary-rgb: #{$primary-rgb};
   --secondary: #{$secondary};

decidim-module-catcomu_managers/app/models/decidim/catcomu_managers/user_override.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Decidim
   module CatcomuManagers
     module UserOverride

decidim-module-catcomu_managers/app/packs/src/decidim/catcomu_managers/admin/managers.js

@@ -1,11 +1,12 @@
 $(() => {
-  $(".civicrm-sync-group-edit").on("click", (e) => {
-    e.preventDefault();
-    $(e.delegateTarget).prev().attr("disabled", false);
+  $(".civicrm-sync-group-edit").on("click", (event) => {
+    event.preventDefault();
+    $(event.delegateTarget).prev().attr("disabled", false);
   });
 
-  $(".civicrm-sync-group-select").on("change", (e) => {
-    let $select = $(e.delegateTarget);
+  $(".civicrm-sync-group-select").on("change", (event) => {
+    let $select = $(event.delegateTarget);
+    /* eslint-disable camelcase */
     $.ajax({
       url: "/admin/managers/scoped_admins",
       data: {

decidim-module-catcomu_managers/app/packs/stylesheets/decidim/catcomu_managers/admin/managers.scss

@@ -1,10 +1,12 @@
 .civicrm-sync-group {
   display: flex;
   align-items: baseline;
+
   &-select {
     margin-right: 4px;
     margin-bottom: 0;
   }
+
   &-edit {
     visibility: hidden;
   }