-
Notifications
You must be signed in to change notification settings - Fork 27
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
Add option to output to Markdown table #69
Comments
Hello, thanks for your suggestion. I like the idea, I don't think implementing it would be too complicated. We could either change the default output or just add an option to allow the choice of the desired output. Would you like to give it a try? |
Changing the default output to a Markdown table wouldn't look pretty in the console. So I'm in favor of adding an option, although I'm not sure how to pass that option to the plugin without having to pass a flag everytime. |
Indeed, let's have another optional table output instead 👍 Speaking of it, which columns do you need? It would be better to have the same columns for the three operations (install, update, uninstall). About the architecture, I think we could add support for a new option in the plugin section of the composer.json. See this documentation to have a look at the existing config supported by this plugin with the I guess we could add an Does this sounds good to you? |
Yes, sounds good! The table columns would preferably be package name, previous version, current version, release notes link, compare link. So something like this; https://github.com/davidrjonas/composer-lock-diff#example-plain-text-table-output |
@pyrech come to think of it, your suggestion would mean I'd have to edit the |
Yes, that is supported 🙂. The plugin configuration can also be set in your global location (usually in |
@pyrech of course, forgot about that one. Perfect! |
We use Gitlab instead of Github, and Gitlab uses markdown for issues. It would be great to have an option to have the output of this plugin generated as a Markdown table, so we can copy/paste it into an issue.
The text was updated successfully, but these errors were encountered: