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

New reporting view for web & add spex version attribute to stage 1 and 2 docs #52

Merged
merged 2 commits into from
Oct 16, 2024

Conversation

mbrsamsung
Copy link
Collaborator

Builds upon the existing report view but now includes the html part of the figures that have linting violations.

Further there was an issue with showing a limited amount of linting errors. The functionality that should show a list of linting errors, was iterating through the list of parsed figures and appending its linting errors. This means that tables that are not being parsed by error or otherwise would not be shown. Now It iterates through the errors and show that.

@mbrsamsung mbrsamsung force-pushed the mbr/feat-ui branch 3 times, most recently from f9c0ee0 to a854a6d Compare October 16, 2024 11:01
Builds upon the existing report view but now includes the html part of
the figures that have linting violations.

Further there was an issue with showing a limited amount of linting
errors. The functionality that should show a list of linting errors, was
iterating through the list of parsed figures and appending its linting
errors. This means that tables that are not being parsed by error or
otherwise would not be shown.  Now It iterates through the errors and
show that.

Signed-off-by: Morten B. Rasmussen <m.rasmussen@samsung.com>
Signed-off-by: Morten B. Rasmussen <m.rasmussen@samsung.com>
@mbrsamsung mbrsamsung changed the title feat(ui): new reporting view for linting errors New reporting view for web & add spex version attribute to stage 1 and 2 docs Oct 16, 2024
@mbrsamsung mbrsamsung merged commit 0d20f89 into main Oct 16, 2024
4 checks passed
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

Successfully merging this pull request may close these issues.

1 participant