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

Ignoring and copying blade.php files #36

Open
alexjacobsma opened this issue Oct 22, 2024 · 5 comments · Fixed by #37
Open

Ignoring and copying blade.php files #36

alexjacobsma opened this issue Oct 22, 2024 · 5 comments · Fixed by #37
Assignees
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@alexjacobsma
Copy link

Is it possible to use this action to encode, but ignore files of a certain type in the directories specified? Ioncube has the ability to ignore and copy. Something like this (on windows), notice the ignore and copy params.

./ioncube_encoder82.exe --ignore '.blade.php' --copy '.blade.php' C:\Github\laravel_app\modules -o C:\Github\laravel_app\modules --replace-target --allow-reflection-all

Project structure I am working with-
modules
users
Controllers
Models
Views
doors
Controllers
Models
Views
ex.

What I want to do is encode all .php files in modules directory while copying blade.php files. Is this possible using this action? I may just be missing it.

@MuriloChianfa
Copy link
Owner

its not possible yet, but we can add this option.

@MuriloChianfa MuriloChianfa self-assigned this Oct 22, 2024
@MuriloChianfa MuriloChianfa added enhancement New feature or request good first issue Good for newcomers labels Oct 22, 2024
@alexjacobsma
Copy link
Author

That is great! Please let me know if I can assist with testing.

@MuriloChianfa
Copy link
Owner

@alexjacobsma can you test the new features in the new version? v2.1.0
you can use the trial version of the ioncube to test.

MuriloChianfa/ioncube-encoder-action@v2.1.0

@alexjacobsma
Copy link
Author

It works! Both copy and ignore were properly passed into the command.

Only one thing that I noticed in two places.

  1. Copy and Ignore Are not recognized as valid Action Inputs.
    image

  2. Same thing here in the warning-
    image

But functionally, it works. Thank you!

@MuriloChianfa
Copy link
Owner

Sorry for the inconvenient, i forgot to add the new options into the allowed inputs list.
I created a new release v2.1.1 to solve this problem.

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

Successfully merging a pull request may close this issue.

2 participants