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

Job names cannot handle formatting properly with reusable workflow inputs #153

Closed
KitsuneAlex opened this issue Aug 20, 2023 · 9 comments · Fixed by #170
Closed

Job names cannot handle formatting properly with reusable workflow inputs #153

KitsuneAlex opened this issue Aug 20, 2023 · 9 comments · Fixed by #170
Labels
enhancement New feature or request

Comments

@KitsuneAlex
Copy link

A small cosmetic issue i found regarding the name formatting of jobs.
When i use string interpolation with input properties, it completely breaks down and looks like this:
image

@KitsuneAlex
Copy link
Author

Is it possible that this is caused by an incompatible Node version by any chance?

@ChristopherHX
Copy link
Owner

Is it possible that this is caused by an incompatible Node version by any chance?

No, it has not been implemented yet. Expression are not evaluated by nodejs, uses nektos/act to do it.

@KitsuneAlex
Copy link
Author

KitsuneAlex commented Aug 20, 2023

Oh, gotcha! ^^ Should i close this for now or just leave it open as it is just a cosmetic issue? :)

@ChristopherHX ChristopherHX added the enhancement New feature or request label Aug 20, 2023
@ChristopherHX
Copy link
Owner

I consider this once I have fun implementing this, until I would keep it open.

@KitsuneAlex
Copy link
Author

KitsuneAlex commented Aug 21, 2023

Sure thing ^^ If you want a helping hand by the way, i'd be happy to help if needed since i really like this runner implementation. I didn't really do much Go before, but i'm an experienced C++ coder and i also got quite comfortable with Rust, so i think i could wrap my head around Go ^^

@ChristopherHX
Copy link
Owner

If you want a helping hand by the way, i'd be happy to help if needed since i really like this runner implementation

I appreciate anyone who want to contribute. I see we are both from Germany, small world. We can also move this project to a GitHub Organization once it make sense.

Actually I just added a long list of known limitation instead of implementing everything. The protocol has been pretty stable between 2021-2023, the first really breaking change is coming soon #96 and there is still some work to do. The most challenging is missing access to the coming server implementation of the new protocol.

I didn't really do much Go before, but i'm an experienced C++ coder

Similar to you, I never touched golang before recycling nektos/act in 2021 to build a xplat runner. I could have written this in c++, but I have choosen go to reuse existing code for running actions locally and cross compiling is much easier than via c++.

@KitsuneAlex
Copy link
Author

Fair enough. And yeah, didn't know from just your name that you're german :D

@KitsuneAlex
Copy link
Author

Do you use nektos/act as a library only? Because it looks very incomplete and unusable for platforms other than Ubuntu right now.

@ChristopherHX
Copy link
Owner

ChristopherHX commented Aug 27, 2023

Do you use nektos/act as a library only?

I use my fork of nektos/act and use it to execute the job (skipping the regular workflow management code), some features of this runner depends on changes of my fork.

But I have written the rest of the code so we can get rid of nekos/act.

it looks very incomplete and unusable for platforms other than Ubuntu right now.

Do you know the HostEnvironment toggle -P <label>=-self-hosted? I have backported this to nektos/act. In 2021 it was impossible to use it without docker backend, I changed that on my own. The documentation about this enhancement of act has not been written yet.

For a sample of a custom worker see: https://github.com/ChristopherHX/megascaler/blob/2acc5d494274426efce35d1cf25bbb51167e96f3/main.go#L84C16-L129 (100% free of nektos/act).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants