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

Jinja2 missing dependecies added and set-ouptut command deprecation warning resolved #3

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

niwciu
Copy link

@niwciu niwciu commented Sep 4, 2024

Describe your changes

  • Added jinja2 dependencies required to generate .html output file
  • set-output command refactor - command deprecation warning resolved

Issue ticket number and link

when set Output file option runner fail #2

Checklist before requesting a review

  • I have performed a self-review of my code
  • If it is a core feature, I have added thorough tests.
  • Do we need to implement analytics?
  • Will this be part of a product update? If yes, please write one phrase about this update.

Added Jinja2 dependencies required to generate .html output file and set-output command deprecation warning resolved

@niwciu
Copy link
Author

niwciu commented Sep 18, 2024

Hi.
Are you planning to look at the proposed changes?
I was planning to use Your actions in my embedded projects automation, but due to the warnings and missing dependencies to generate HTML output, I was not able to cover all needs. That was the reason why this pull request was created. I'm asking because I prefer to use officially your action and show that it's your work rather than make a copy in my repo and link to it in workflows.
Best Regards

@aleksei-shabalin-simplisafe

It will be nice to merge it to get rid of that annoying warning about set-output

@niwciu
Copy link
Author

niwciu commented Dec 10, 2024

It will be nice to merge it to get rid of that annoying warning about set-output

It looks like it's not gone happen.
If you want to get rid of this warning you can use my fork.
Just replace your current "uses" parameter with

uses: niwciu/Lizard-Runner@develop

I will most likely create my own action for this tool (lizard) soon. If you are interested, just let me know. I will inform you when it's available.

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