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

Allow Skunk to HTML output the results it generates #50

Open
bronzdoc opened this issue Oct 5, 2020 · 1 comment
Open

Allow Skunk to HTML output the results it generates #50

bronzdoc opened this issue Oct 5, 2020 · 1 comment

Comments

@bronzdoc
Copy link
Contributor

bronzdoc commented Oct 5, 2020

Right now skunk will display the output it generates as simple text.
We would like that skunk could take a --format flag where you can specify an html format

We probaby want to abstract this code into its own class so we are able to switch formatters at runtime

@bronzdoc bronzdoc added the ROSS ROSSConf 2020 label Oct 5, 2020
@etagwerker
Copy link
Member

Considering that Skunk is an extension of RubyCritic, this could be added following the same pattern that RubyCritic (RC) implements.

I don't think this would require any changes in RubyCritic. It could be generated as an additional section in the RC HTML report: https://github.com/whitesmith/rubycritic#overview

@bronzdoc bronzdoc added Feature Request and removed ROSS ROSSConf 2020 labels Aug 11, 2021
@JuanVqz JuanVqz mentioned this issue Feb 7, 2023
1 task
JuanVqz added a commit that referenced this issue Feb 7, 2023
As first step we need to move the console report away of the status
reporter class.

Related #50
JuanVqz added a commit that referenced this issue Feb 10, 2023
As first step we need to move the console report away of the status
reporter class.

Related #50
JuanVqz added a commit that referenced this issue Feb 11, 2023
As first step we need to move the console report away of the status
reporter class.

Related #50
etagwerker pushed a commit that referenced this issue Jul 20, 2023
As first step we need to move the console report away of the status
reporter class.

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

No branches or pull requests

2 participants