Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in
  • E escet
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Graph
    • Compare
    • Locked Files
  • Issues 93
    • Issues 93
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
  • Merge requests 3
    • Merge requests 3
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
    • Test Cases
  • Deployments
    • Deployments
    • Releases
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Eclipse ProjectsEclipse Projects
  • Eclipse ESCET (Supervisory Control Engineering Toolkit)
  • escet
  • Merge requests
  • !464

Draft: #454 New output for CIF checks

  • Review changes

  • Download
  • Email patches
  • Plain diff
Closed Dennis Hendriks requested to merge 454-reconsider-output-of-cif-checks into develop Jan 01, 2023
  • Overview 28
  • Commits 35
  • Pipelines 7
  • Changes 292
  • This merge request is not ready by any means. It serves to allow discussion on this attempt at new output.
  • I had to move all CIF checks to a new plugin to prevent a cyclic dependency.
  • The design for checks is much simpler, as there are no message classes anymore. Just (formatted) strings.
  • I got rid of reporting on ancestors or named objects, requiring PositionObject objects with a position. It turns out not all expressions and types have a position, even after re-parsing. I haven't looked into why that is yet. Let's see what we want as output first.
  • No more reporting on null specifications. Just supply the Specification object instead. Typically, this is used in preprocessSpecification or postprocessSpecification, which have that already anyway. Can use CifScopeUtils.getSpecification otherwise.
  • For the new output, I opted to first categorize by message, and then sort per line, and then for all violations on the same line report them with the text of the line and markers.

Addresses #454 (closed)

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: 454-reconsider-output-of-cif-checks

Copyright © Eclipse Foundation, Inc. All Rights Reserved.     Privacy Policy | Terms of Use | Copyright Agent