Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Mis-characterisation of SVG and image as generated #9

Open
hoijui opened this issue Feb 25, 2022 · 2 comments
Open

Mis-characterisation of SVG and image as generated #9

hoijui opened this issue Feb 25, 2022 · 2 comments

Comments

@hoijui
Copy link
Owner

hoijui commented Feb 25, 2022

In GitLab by @julianstirling on Feb 25, 2022, 17:27

SVG is the native file format of Inkscape and is a valid source file. It should not be flagged as a generated file.

Similarly jpg files are generally photos which if they are in documentation the cannot be generated by a CI. Similarly some PNGs (for example high resolution renders) also generally takes specialist computing hardware and cannot be generated on-the-fly by a CI.

@hoijui
Copy link
Owner Author

hoijui commented May 30, 2022

I am a bit twisted about this one. I think it probably needs adjustment in the way it is reported. Right now (sample):


  • Might be generated
    • LIGHT - Possibly generatable files found. Please consider removing them:

      • res/asset/media/img/3D-printer.png
      • res/asset/media/img/x-y-control-plan.png

This shows that this is only a minor thing (with "LIGHT"),
but it still shows the check as failed (no tick in checkbox).
That is indeed not optimal. Possible ways to improve:

  1. Only fail (no tick in checkbox) for issues above level "LIGHT"
  2. instead of boolean check states (passed/not-passed), introduce more (named) states, or a value from 0 to 10 (or 0.0 to 1.0)
  3. Use solution 1., but allow to enable --pedantic mode, which would result in the current behavior
  4. ... any other ideas?

@hoijui
Copy link
Owner Author

hoijui commented May 30, 2022

marked this issue as related to #12

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant