Add the ability to add ignored directories in editor and commandlet

[eddiebeazer]

[d-franklin]

Thanks for your work on getting this working with UE5.

I'm wondering if ignoring folders is an option in the current version or it's still a work in progress?

[eddiebeazer]

@d-franklin I think it's still possible. I have my own edited version of the plugin where I'm doing it manually. I believe the only thing stopping me from adding it to the main branch is just that I need to figure out how to add a menu to the Project Settings UI so each game can specify and save the settings in a .ini.

When I review your PR I'll try to take a look at it. I believe it shouldn't be all too bad considering that there is already UI options for specifying the type of linter

[d-franklin]

@d-franklin I think it's still possible. I have my own edited version of the plugin where I'm doing it manually. I believe the only thing stopping me from adding it to the main branch is just that I need to figure out how to add a menu to the Project Settings UI so each game can specify and save the settings in a .ini.

Even having a manual option would be nice. The setting menu is preferable but any way to exclude folders is good, especially when using marketplace assets since the linter will throw a fit.

When I review your PR I'll try to take a look at it. I believe it shouldn't be all too bad considering that there is already UI options for specifying the type of linter

RE my PR. It's nothing special, it just removes one line (in the command line version) that is adding the Game folder as the default scan folder, it's being run twice when it only needs to run once. But in it's current state doesn't effect anything.

[eddiebeazer]

Do you run Linter with the cli or in the editor? When I added the manual paths I did it for cli usage for my CI workflow. I’m fairly busy the next week or two but I’ll see if I can get to this by the end of the month. I’ll add a way to specify folders and files (I ignore my ABP due to a max node function issue I can’t do anything about).

…

On Sep 6, 2024 at 5:46 AM -0400, DFranklin ***@***.***>, wrote: > @d-franklin I think it's still possible. I have my own edited version of the plugin where I'm doing it manually. I believe the only thing stopping me from adding it to the main branch is just that I need to figure out how to add a menu to the Project Settings UI so each game can specify and save the settings in a .ini. Even having a manual option would be nice. The setting menu is preferable but any way to exclude folders is good, especially when using marketplace assets since the linter will throw a fit. > When I review your PR I'll try to take a look at it. I believe it shouldn't be all too bad considering that there is already UI options for specifying the type of linter RE my PR. It's nothing special, it just removes one line (in the command line version) that is adding the Game folder as the default scan folder, it's being run twice when it only needs to run once. But in it's current state doesn't effect anything. — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you were assigned.Message ID: ***@***.***>

[d-franklin]

Do you run Linter with the cli or in the editor? When I added the manual paths I did it for cli usage for my CI workflow.

Both. The editor allows for correcting mistakes before they get to source control and/or CI and the cli catches human error. There is always the option of running the cli before commits to SC but the editor just makes life easier.

I’m fairly busy the next week or two but I’ll see if I can get to this by the end of the month.

Cool, no rush.