Skip to content

Commit

Permalink
#221 yarn lint + format (#223)
Browse files Browse the repository at this point in the history
* #221 yarn lint + format

* #221 Align pre-commit lint and format with scripts

---------

Co-authored-by: wouter-adriaens <[email protected]>
  • Loading branch information
yannickkuypers and wouter-adriaens authored Feb 20, 2024
1 parent 82da089 commit dc5dbd7
Show file tree
Hide file tree
Showing 33 changed files with 65 additions and 76 deletions.
31 changes: 12 additions & 19 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,26 +5,19 @@ repos:
- id: check-yaml
- id: end-of-file-fixer
- id: trailing-whitespace
- repo: https://github.com/pre-commit/mirrors-prettier
rev: v3.0.0-alpha.9-for-vscode
- repo: local
hooks:
- id: prettier
additional_dependencies:
- [email protected]
name: run format
language: system
entry: bash -c 'yarn format'
pass_filenames: false
types_or: [ts, javascript, scss, css, yaml, json, vue]
- repo: https://github.com/pre-commit/mirrors-eslint
rev: v8.56.0
- repo: local
hooks:
- id: eslint
additional_dependencies:
- [email protected]
- [email protected]
- [email protected]
- [email protected]
- [email protected]
- [email protected]
- [email protected]
- '@vue/[email protected]'
- '@vue/[email protected]'
files: \.[jt]sx|\.vue?$ # *.js, *.jsx, *.ts, *.tsx and *.vue
types: [file]
- id: lint
name: run lint
language: system
entry: bash -c 'yarn lint'
pass_filenames: false
types_or: [ts, javascript, scss, css, yaml, json, vue]
2 changes: 1 addition & 1 deletion src/__tests__/InputPhone.cy.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import type { CountryCode } from 'libphonenumber-js';
import { defineComponent, ref } from 'vue';
import InputPhone from '@components/dumb/InputPhone.vue';
import type { CountryCode } from 'libphonenumber-js';

const TestComponent = defineComponent({
components: { InputPhone },
Expand Down
2 changes: 1 addition & 1 deletion src/__tests__/OeZoneerder.cy.ts
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
/* eslint-disable vue/one-component-per-file */
import { defineComponent } from 'vue';
import { LayerType } from '@/models';
import type { OeZoneerderProps } from '@/models';
import { OeZoneerder } from '@components/smart';
import type { OeZoneerderProps } from '@/models';

describe('OeZoneerder', () => {
describe('default', () => {
Expand Down
8 changes: 4 additions & 4 deletions src/components/dumb/OeInventarisLoadedLink.vue
Original file line number Diff line number Diff line change
Expand Up @@ -3,11 +3,11 @@
<template v-if="props.waarnemingen?.length > 0">
<h5 class="vl-title vl-title--h5" data-cy="gekoppelde-waarnemingen-title">Gekoppelde waarnemingen</h5>
<a
v-for="waarneming of props.waarnemingen"
:key="waarneming.id"
class="vl-u-display-block"
target="_blank"
v-for="waarneming of props.waarnemingen"
:data-cy="'waarneming-link-' + waarneming.id"
:key="waarneming.id"
:href="waarneming.uri"
>{{ waarneming.naam }} ({{ waarneming.id }})</a
>
Expand All @@ -20,10 +20,10 @@
<template v-if="props.gebeurtenissen?.length > 0">
<h5 class="vl-title vl-title--h5" data-cy="gekoppelde-gebeurtenissen-title">Gekoppelde gebeurtenissen</h5>
<a
class="vl-u-display-block"
target="_blank"
v-for="gebeurtenis of props.gebeurtenissen"
:key="gebeurtenis.id"
class="vl-u-display-block"
target="_blank"
:href="gebeurtenis.uri"
:data-cy="'gebeurtenis-link-' + gebeurtenis.id"
>{{ gebeurtenis.titel }} ({{ gebeurtenis.id }})</a
Expand Down
2 changes: 1 addition & 1 deletion src/components/smart/FilterAOEActor.vue
Original file line number Diff line number Diff line change
Expand Up @@ -11,10 +11,10 @@
<script setup lang="ts">
import OeAutocomplete from '../dumb/OeAutocomplete.vue';
import { computed, ref } from 'vue';
import { ActorService } from '@services/actor.service';
import type { IActor } from '@models/actor';
import type { IAutocompleteOption } from '@models/autocomplete';
import type { IFilterActorProps } from '@models/index';
import { ActorService } from '@services/actor.service';
const props = withDefaults(defineProps<IFilterActorProps>(), {
id: '',
Expand Down
2 changes: 1 addition & 1 deletion src/components/smart/FilterAanduidingsobject.vue
Original file line number Diff line number Diff line change
Expand Up @@ -13,9 +13,9 @@
import OeAutocomplete from '../dumb/OeAutocomplete.vue';
import { toRef } from '@vueuse/core';
import { ref, watch } from 'vue';
import { InventarisApiService } from '@services/inventaris-api.service';
import type { IAutocompleteOption } from '@models/autocomplete';
import type { IESAanduidingsobject, IFilterAanduidingsobjectProps } from '@models/index';
import { InventarisApiService } from '@services/inventaris-api.service';
const props = withDefaults(defineProps<IFilterAanduidingsobjectProps>(), {
id: '',
Expand Down
2 changes: 1 addition & 1 deletion src/components/smart/FilterGemeente.vue
Original file line number Diff line number Diff line change
Expand Up @@ -22,9 +22,9 @@
<script setup lang="ts">
import { VlMultiselect } from '@govflanders/vl-ui-design-system-vue3';
import { computed, onBeforeMount, ref } from 'vue';
import { CrabApiService } from '@services/crab-api.service';
import type { IFilterGemeenteProps } from '@models/index';
import type { IGemeente } from '@models/locatie';
import { CrabApiService } from '@services/crab-api.service';
const props = withDefaults(defineProps<IFilterGemeenteProps>(), {
api: '',
Expand Down
2 changes: 1 addition & 1 deletion src/components/smart/FilterProvincie.vue
Original file line number Diff line number Diff line change
Expand Up @@ -22,9 +22,9 @@
<script setup lang="ts">
import { VlMultiselect } from '@govflanders/vl-ui-design-system-vue3';
import { computed, onBeforeMount, ref } from 'vue';
import { CrabApiService } from '@services/crab-api.service';
import type { IFilterProvincieProps } from '@models/index';
import type { IProvincie } from '@models/locatie';
import { CrabApiService } from '@services/crab-api.service';
const props = withDefaults(defineProps<IFilterProvincieProps>(), {
api: '',
Expand Down
2 changes: 1 addition & 1 deletion src/components/smart/OeActorWidget.vue
Original file line number Diff line number Diff line change
Expand Up @@ -42,8 +42,8 @@ import { ref } from 'vue';
import OeLoader from '@components/dumb/OeLoader.vue';
import Detail from '@components/smart/OeActorWidgetDetail.vue';
import Grid from '@components/smart/OeActorWidgetGrid.vue';
import type { IActor } from '@models/actor';
import { ActorService } from '@services/actor.service';
import type { IActor } from '@models/actor';
interface IOeActorWidgetProps {
id: string;
Expand Down
4 changes: 2 additions & 2 deletions src/components/smart/OeActorWidgetGrid.vue
Original file line number Diff line number Diff line change
Expand Up @@ -43,11 +43,11 @@
import OeGrid from '../dumb/OeGrid.vue';
import OeActorWidgetGridActies from './OeActorWidgetGridActies.vue';
import { VlButton, VlSearch } from '@govflanders/vl-ui-design-system-vue3';
import type { ColDef, FirstDataRenderedEvent, GridOptions, IGetRowsParams, RowClickedEvent } from 'ag-grid-community';
import { isEmpty, omitBy } from 'lodash';
import { computed, getCurrentInstance, ref } from 'vue';
import type { IActor } from '@models/actor';
import { ActorService, type IActorenQuery } from '@services/actor.service';
import type { ColDef, FirstDataRenderedEvent, GridOptions, IGetRowsParams, RowClickedEvent } from 'ag-grid-community';
import type { IActor } from '@models/actor';
interface IOeActorWidgetGridProps {
api: string;
Expand Down
12 changes: 6 additions & 6 deletions src/components/smart/OeZoneerder.vue
Original file line number Diff line number Diff line change
Expand Up @@ -37,28 +37,28 @@ import { FontAwesomeIcon } from '@fortawesome/vue-fontawesome';
import Map from 'ol/Map';
import View from 'ol/View';
import { Attribution, Control, FullScreen, Rotate, ScaleLine, Zoom, ZoomToExtent } from 'ol/control';
import type { Coordinate } from 'ol/coordinate';
import { getCenter, getTopLeft, getWidth } from 'ol/extent';
import type { Extent } from 'ol/extent';
import Point from 'ol/geom/Point';
import { Group, Layer, Tile } from 'ol/layer';
import { type ProjectionLike, get as getOlProj, transformExtent } from 'ol/proj';
import type { Projection } from 'ol/proj';
import { register } from 'ol/proj/proj4';
import { TileWMS, WMTS } from 'ol/source';
import WMTSTileGrid from 'ol/tilegrid/WMTS';
import proj4 from 'proj4';
import { onMounted, onUnmounted, provide, ref, watch } from 'vue';
import type { Ref } from 'vue';
import { LayerType, defaultControlConfig, defaultLayerConfig } from '@/models';
import type { IBoundingBox, LayerOptions, OeZoneerderProps } from '@/models';
import { CrabApiService } from '@/services';
import OeAutocomplete from '@components/dumb/OeAutocomplete.vue';
import Layerswitcher from '@components/smart/OeZoneerderLayerswitcher.vue';
import ZonePanel from '@components/smart/OeZoneerderZonePanel.vue';
import { Geolocate } from '@utils/openlayers/oe-ol-geolocate';
import type { Coordinate } from 'ol/coordinate';
import type { Extent } from 'ol/extent';
import type { Projection } from 'ol/proj';
import type { Ref } from 'vue';
import type { IBoundingBox, LayerOptions, OeZoneerderProps } from '@/models';
import type { IAutocompleteOption } from '@models/autocomplete';
import type { Contour } from '@models/oe-openlayers';
import { Geolocate } from '@utils/openlayers/oe-ol-geolocate';

const props = withDefaults(defineProps<OeZoneerderProps>(), {
controlConfig: () => defaultControlConfig,
Expand Down
6 changes: 3 additions & 3 deletions src/components/smart/OeZoneerderLayerswitcher.vue
Original file line number Diff line number Diff line change
Expand Up @@ -52,14 +52,14 @@
import 'ol/ol.css';
import { FontAwesomeIcon } from '@fortawesome/vue-fontawesome';
import { VlCheckbox, VlRadio, VlTitle } from '@govflanders/vl-ui-design-system-vue3';
import type Map from 'ol/Map';
import { Group } from 'ol/layer';
import type BaseLayer from 'ol/layer/Base';
import { v4 as uuidv4 } from 'uuid';
import { inject, onMounted, onUnmounted, ref, watch } from 'vue';
import { vClickOutside } from '@directives/click-outside.directive';
import type Map from 'ol/Map';
import type BaseLayer from 'ol/layer/Base';
import type { Ref } from 'vue';
import type { CrabApiService } from '@/services';
import { vClickOutside } from '@directives/click-outside.directive';
const buttonRef = ref<HTMLElement>();
const elementRef = ref<HTMLElement>();
Expand Down
8 changes: 4 additions & 4 deletions src/components/smart/OeZoneerderZonePanel.vue
Original file line number Diff line number Diff line change
Expand Up @@ -83,10 +83,6 @@ import 'ol/ol.css';
import { FontAwesomeIcon } from '@fortawesome/vue-fontawesome';
import { VlButton, VlIcon, VlInputField, VlInputGroup, VlLink, VlTitle } from '@govflanders/vl-ui-design-system-vue3';
import { Feature, Map, MapBrowserEvent } from 'ol';
import type { FeatureLike } from 'ol/Feature';
import type { ColorLike } from 'ol/colorlike';
import type { Listener } from 'ol/events';
import type { Extent } from 'ol/extent';
import { GeoJSON, WKT } from 'ol/format';
import { Circle, Geometry, MultiPolygon, Polygon } from 'ol/geom';
import { fromCircle } from 'ol/geom/Polygon';
Expand All @@ -95,6 +91,10 @@ import VectorLayer from 'ol/layer/Vector';
import VectorSource from 'ol/source/Vector';
import { Fill, Text as OlText, Stroke, Style } from 'ol/style';
import { inject, onMounted, onUnmounted, ref, watch } from 'vue';
import type { FeatureLike } from 'ol/Feature';
import type { ColorLike } from 'ol/colorlike';
import type { Listener } from 'ol/events';
import type { Extent } from 'ol/extent';
import type { CrabApiService } from '@/services';
import type { Contour, IDrawGeomType } from '@models/oe-openlayers';
Expand Down
2 changes: 1 addition & 1 deletion src/models/aanduidingsobject.ts
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
import type { IDataverantwoordelijke } from './dataverantwoordelijke';
import { Serializable } from './serializable';
import type { IDataverantwoordelijke } from './dataverantwoordelijke';
import type { IStatus } from './status';
import type { ISystemFields } from './system-fields';

Expand Down
12 changes: 4 additions & 8 deletions src/scss/_override-fonts.scss
Original file line number Diff line number Diff line change
@@ -1,34 +1,30 @@
@font-face {
font-family: 'Flanders Art Sans';
src:
url('/node_modules/pyoes/fonts/FlandersArtSans-Light.ttf') format('truetype'),
src: url('/node_modules/pyoes/fonts/FlandersArtSans-Light.ttf') format('truetype'),
url('/node_modules/pyoes/fonts/FlandersArtSans-Light.woff') format('woff');
font-weight: 300;
font-style: normal;
}

@font-face {
font-family: 'Flanders Art Sans';
src:
url('/node_modules/pyoes/fonts/FlandersArtSans-Regular.ttf') format('truetype'),
src: url('/node_modules/pyoes/fonts/FlandersArtSans-Regular.ttf') format('truetype'),
url('/node_modules/pyoes/fonts/FlandersArtSans-Regular.woff') format('woff');
font-weight: 400;
font-style: normal;
}

@font-face {
font-family: 'Flanders Art Sans';
src:
url('/node_modules/pyoes/fonts/FlandersArtSans-Medium.ttf') format('truetype'),
src: url('/node_modules/pyoes/fonts/FlandersArtSans-Medium.ttf') format('truetype'),
url('/node_modules/pyoes/fonts/FlandersArtSans-Medium.woff') format('woff');
font-weight: 500;
font-style: normal;
}

@font-face {
font-family: 'Flanders Art Sans';
src:
url('/node_modules/pyoes/fonts/FlandersArtSans-Bold.ttf') format('truetype'),
src: url('/node_modules/pyoes/fonts/FlandersArtSans-Bold.ttf') format('truetype'),
url('/node_modules/pyoes/fonts/FlandersArtSans-Bold.woff') format('woff');
font-weight: 700;
font-style: normal;
Expand Down
4 changes: 2 additions & 2 deletions src/services/crab-api.service.ts
Original file line number Diff line number Diff line change
@@ -1,9 +1,10 @@
import { HttpService } from './http.service';
import { sortBy } from 'lodash';
import type { Coordinate } from 'ol/coordinate';
import WFS from 'ol/format/WFS';
import Intersects from 'ol/format/filter/Intersects';
import Point from 'ol/geom/Point';
import { Niscode } from '@models/niscode.enum';
import type { Coordinate } from 'ol/coordinate';
import type {
IAdres,
IGemeente,
Expand All @@ -15,7 +16,6 @@ import type {
IProvincie,
IStraat,
} from '@models/locatie';
import { Niscode } from '@models/niscode.enum';

export class CrabApiService extends HttpService {
readonly API_URL: string;
Expand Down
2 changes: 1 addition & 1 deletion src/stories/dumb-components/autocomplete.stories.ts
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
import '@/scss/main.scss';
import OeAutocomplete from '../../components/dumb/OeAutocomplete.vue';
import { ref } from 'vue';
import type { IAutocompleteOption } from '../../models/autocomplete';
import type { Meta, StoryObj } from '@storybook/vue3';
import { ref } from 'vue';

const meta: Meta<typeof OeAutocomplete> = {
title: 'Dumb components/Autocomplete',
Expand Down
2 changes: 1 addition & 1 deletion src/stories/dumb-components/button.stories.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import '@/scss/main.scss';
import type { Meta, StoryObj } from '@storybook/vue3';
import OeButton from '@components/dumb/OeButton.vue';
import type { Meta, StoryObj } from '@storybook/vue3';

// More on how to set up stories at: https://storybook.js.org/docs/vue/writing-stories/introduction
const meta: Meta<typeof OeButton> = {
Expand Down
2 changes: 1 addition & 1 deletion src/stories/dumb-components/container.stories.ts
Original file line number Diff line number Diff line change
@@ -1,9 +1,9 @@
import '@/scss/main.scss';
import { VlButton } from '@govflanders/vl-ui-design-system-vue3';
import type { Meta, StoryObj } from '@storybook/vue3';
import { last, remove } from 'lodash';
import { ref } from 'vue';
import OeContainer from '@components/dumb/OeContainer.vue';
import type { Meta, StoryObj } from '@storybook/vue3';
import type { ITab } from '@models/container';

// More on how to set up stories at: https://storybook.js.org/docs/vue/writing-stories/introduction
Expand Down
2 changes: 1 addition & 1 deletion src/stories/dumb-components/filter-radio.stories.ts
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import '@/scss/main.scss';
import FilterRadio from '../../components/dumb/FilterRadio.vue';
import type { Meta, StoryObj } from '@storybook/vue3';
import { ref } from 'vue';
import type { Meta, StoryObj } from '@storybook/vue3';

// More on how to set up stories at: https://storybook.js.org/docs/vue/writing-stories/introduction
const meta: Meta<typeof FilterRadio> = {
Expand Down
2 changes: 1 addition & 1 deletion src/stories/dumb-components/grid.stories.ts
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
import '@/scss/main.scss';
import OeGrid from '@components/dumb/OeGrid.vue';
import type { Meta, StoryObj } from '@storybook/vue3';
import type {
FirstDataRenderedEvent,
Expand All @@ -7,7 +8,6 @@ import type {
ICellRendererParams,
IGetRowsParams,
} from 'ag-grid-community';
import OeGrid from '@components/dumb/OeGrid.vue';

// More on how to set up stories at: https://storybook.js.org/docs/vue/writing-stories/introduction
const meta: Meta<typeof OeGrid> = {
Expand Down
2 changes: 1 addition & 1 deletion src/stories/dumb-components/header.stories.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import '@/scss/main.scss';
import type { Meta, StoryObj } from '@storybook/vue3';
import OeHeader from '@components/dumb/OeHeader.vue';
import type { Meta, StoryObj } from '@storybook/vue3';

// More on how to set up stories at: https://storybook.js.org/docs/vue/writing-stories/introduction
const meta: Meta<typeof OeHeader> = {
Expand Down
2 changes: 1 addition & 1 deletion src/stories/dumb-components/input-phone.stories.ts
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import '@/scss/main.scss';
import type { Meta, StoryObj } from '@storybook/vue3';
import { ref } from 'vue';
import InputPhone from '@components/dumb/InputPhone.vue';
import type { Meta, StoryObj } from '@storybook/vue3';

// More on how to set up stories at: https://storybook.js.org/docs/vue/writing-stories/introduction
const meta: Meta<typeof InputPhone> = {
Expand Down
2 changes: 1 addition & 1 deletion src/stories/dumb-components/loader.stories.ts
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
import '@/scss/main.scss';
import OeLoader from '../../components/dumb/OeLoader.vue';
import { VlButton } from '@govflanders/vl-ui-design-system-vue3';
import type { Meta, StoryObj } from '@storybook/vue3';
import { ref } from 'vue';
import type { Meta, StoryObj } from '@storybook/vue3';

const meta: Meta<typeof OeLoader> = {
title: 'Dumb components/Loader',
Expand Down
2 changes: 1 addition & 1 deletion src/stories/dumb-components/no-rows-overlay.stories.ts
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
import type { Meta, StoryObj } from '@storybook/vue3';
import NoRowsOverlay from '@components/dumb/NoRowsOverlay.vue';
import type { Meta, StoryObj } from '@storybook/vue3';

const meta: Meta<typeof NoRowsOverlay> = {
title: 'Dumb components/NoRowsOverlay',
Expand Down
2 changes: 1 addition & 1 deletion src/stories/dumb-components/select.stories.ts
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import '@/scss/main.scss';
import OeSelect from '../../components/dumb/OeSelect.vue';
import type { Meta, StoryObj } from '@storybook/vue3';
import { ref } from 'vue';
import type { Meta, StoryObj } from '@storybook/vue3';
import type { ISelectOption } from '@models/select';

const meta: Meta<typeof OeSelect> = {
Expand Down
2 changes: 1 addition & 1 deletion src/stories/dumb-components/system-fields.stories.ts
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
import type { Meta, StoryObj } from '@storybook/vue3';
import SystemFields from '@components/dumb/SystemFields.vue';
import type { Meta, StoryObj } from '@storybook/vue3';

const meta: Meta<typeof SystemFields> = {
title: 'Dumb components/SystemFields',
Expand Down
Loading

0 comments on commit dc5dbd7

Please sign in to comment.