Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • D database-obmbuildings
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 0
    • Issues 0
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 0
    • Merge requests 0
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Container Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Dynamic ExposureDynamic Exposure
  • OpenBuildingMapOpenBuildingMap
  • database-obmbuildings
  • Merge requests
  • !6

Added ci

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Marius Kriegerowski requested to merge feature/linter into master Aug 12, 2021
  • Overview 10
  • Commits 1
  • Pipelines 23
  • Changes 1

@fd there are some issues which sqlfluff complains about. Not sure if we should use a different linter or any linter at all but I'd feel better when contributing.

Update: The ci now uses the built artifact from our docker-obm-database pulled from here: https://git.gfz-potsdam.de/dynamicexposure/server-components/containers/docker-obm-database/container_registry It's still using the produced one from the merge request. Just need to update the pulled image as soon as this is in master). It then executes all files in migrations to see if all work.

The linter complaints about a couple of issues and unparseable things. E.g. the 4 space indentation seems invalid. Should can you install the linter and give it a try. I'm not experiences enough in sql to judge if the linter is viable or not.

Edited Aug 16, 2021 by Marius Kriegerowski
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: feature/linter