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

Note support and additional filtered report #9

Merged

Conversation

lucas-koehler
Copy link
Contributor

Improve HTML generator code

  • Refactor the HTML generator to reuse common code and improve code readability.
  • Fix typo in HTML generator
  • Fix indentation and missing semicolons

Support for additional notes in report

Add a parser for additional information parsed from a YAML file.
Notes can be specified for any row in the report via a YAML file.
These are read during generation and added in a new column Note.

Additional filtered report only containing unsupported API

  • Extend the generator to generate a second output file filtered-status.html that only contains unsupported API.
  • Extend comparator with a removeSupported method
  • Generate second report in index.ts
  • Extend README with second badge and update example output

Add initial issue links

Additional minor fixes

  • No longer allow unused variables in tsconfig.json
  • Exit with error code 1 in case an error occurs

Contributed on behalf of STMicroelectronics

Signed-off-by: Lucas Koehler lkoehler@eclipsesource.com

* Refactor the HTML generator to reuse common code and improve code readability.
* Fix typo in HTML generator
* Fix indentation and missing semicolons

Contributed on behalf of STMicroelectronics

Signed-off-by: Lucas Koehler <lkoehler@eclipsesource.com>
Add a parser for additional information parsed from a YAML file.
For now, only a note can be specified.

Notes can be specified for any row in the report via a YAML file.
These are read during generation and added in a new column Note.

Additional minor fixes:
* No longer allow unused variables in tsconfig.json
* Exit with error code 1 in case an error occurs

Contributed on behalf of STMicroelectronics

Signed-off-by: Lucas Koehler <lkoehler@eclipsesource.com>
Extend the generator to generate a second output file filtered-status.html that only contains unsupported API.

* Extend comparator with a removeSupported method
* Generate second report in index.ts
* Extend README with second badge and update example output

Contributed on behalf of STMicroelectronics

Signed-off-by: Lucas Koehler <lkoehler@eclipsesource.com>
* Add initial issue links as notes based on eclipse-theia/theia#9863.
* Tweak generated html code to open clicked links in a new tab/window and improve link color for readability

Contributed on behalf of STMicroelectronics

Signed-off-by: Lucas Koehler <lkoehler@eclipsesource.com>
@JonasHelming JonasHelming merged commit 9c38d6d into eclipse-theia:master Jan 24, 2022
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.

2 participants