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

Improving ccsh check behaviour #3734

Open
3 tasks done
Nereboss opened this issue Sep 12, 2024 · 1 comment
Open
3 tasks done

Improving ccsh check behaviour #3734

Nereboss opened this issue Sep 12, 2024 · 1 comment
Labels
cc.json Everything that modifies the cc.json structure. NOT for Importers that do NOT add new fields to json pr-analysis Issues that touch the analysis pr(oject). question UX / UI Everything related to design and user experience

Comments

@Nereboss
Copy link
Collaborator

Question

Preparation

Please check these resources first before posting a question

Question

Should we improve the behaviour of ccsh check?
I find it to be not the most intuitive right now:

  • When the input is a valid file, it simply runs and exits without any additional information. This can leave the user guessing if it even ran correctly as there is no feedback
  • When the input is an invalid file, it prints an exception containing what was wrong. At first sight this looks like the program ran into an error and not, that it successfully ran and concluded that the file is invalid.

How often is this functionality used? Would it be worth it to improve its output to more clearly state its findings?

@Nereboss Nereboss added question pr-analysis Issues that touch the analysis pr(oject). cc.json Everything that modifies the cc.json structure. NOT for Importers that do NOT add new fields to json UX / UI Everything related to design and user experience labels Sep 12, 2024
@ChristianHuehn
Copy link
Collaborator

I can say that in the last two audits I attended, there werent any usages of ccsh check. I will ask around 👍 thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cc.json Everything that modifies the cc.json structure. NOT for Importers that do NOT add new fields to json pr-analysis Issues that touch the analysis pr(oject). question UX / UI Everything related to design and user experience
Projects
None yet
Development

No branches or pull requests

2 participants