Skip to content

polish code, fix vulnerabilities, update main README

Diego Molteni requested to merge slb/dm3/polishing-code into master

Type of change

  • Bug Fix
  • Feature
  • Pipeline
  • Test
  • Documentation
  • Code Style Review

Does this introduce a change in the core logic?

  • No
  • Yes

Does this introduce a change in the cloud provider implementation, if so which cloud?

  • Azure

Does this follow conventional commits spec?

  • No
  • Yes

Have you set the target Milestone?

  • No
  • Yes

Have you set the no-detached-pipeline label?

  • No
  • Yes

Updates description?

  • Added code speller and fixed typos and spelling errors in code
  • Reviewed linter an fixed lint issue in code
  • Added pre commit hook to run code spell and lint
  -------------------------------------------------------
  1 - spelling errors check
  -------------------------------------------------------
  
  > sdms@4.0.0 spell
  > cspell --no-progress .

  CSpell: Files checked: 148, Issues found: 0 in 0 files

  -------------------------------------------------------
  2 - code linting
  -------------------------------------------------------

  > sdms@4.0.0 lint
  > eslint --ignore-path .eslintignore --ext .js,.ts .

  ------------------------------------------------------
  • Added missing headers info
  • Upgraded dependencies to the latest version (fixed code vulnerabilities)
    • @azure/core-rest-pipeline ^1.10.1 → ^1.11.0
    • @azure/identity ^3.1.4 → ^3.2.3
    • @types/express ^4.17.15 → ^4.17.17
    • @types/node ^18.11.18 → ^20.4.1
    • axios ^1.2.2 → ^1.4.0
    • ioredis ^5.2.4 → ^5.3.2
    • qs ^6.11.0 → ^6.11.2
    • swagger-ui-express ^4.6.0 → ^5.0.0
    • typescript ^4.9.4 → ^5.1.6
  • Updated docker README
  • Created main README
  • Created cli README
  • Reviewed cli generation scripts
Edited by Diego Molteni

Merge request reports