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

Configure the namespace of the policy file #313

Merged
merged 2 commits into from
Feb 14, 2024

Conversation

igwen6w
Copy link
Contributor

@igwen6w igwen6w commented Jan 16, 2024

Since you can configure the path of the policy file, you must also be able to configure the namespace of the policy file.

Copy link

what-the-diff bot commented Jan 16, 2024

PR Summary

  • Addition of a new 'namespace' entry
    A new 'namespace' entry has been introduced under 'generator' in one of the configuration files. This will help define the scope under which certain codes belong, enhancing their organization and understanding.

  • Update of 'CanGeneratePolicy.php' file
    The 'CanGeneratePolicy.php' file has been updated with improvements aimed at better namespaces handling. This update primarily involves the use of a new method called 'getGeneratorNamespace()' that would now be used for the 'namespace' field when the file path contains certain elements like 'vendor' or 'src'. This method has also replaced previous usage of 'Models' for the 'namespace' field.

  • Introduction of getGeneratorNamespace() method
    A new method called 'getGeneratorNamespace()' has been added to a utility file. The purpose of this method is to fetch the relevant namespace for a generator, providing improved convenience and readability for developers.

@bezhanSalleh bezhanSalleh merged commit 95fa408 into bezhanSalleh:3.x Feb 14, 2024
1 check passed
@bezhanSalleh
Copy link
Owner

Thanks

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.

None yet

2 participants