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

add modules in nginx.conf #39

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

seonghobae
Copy link
Contributor

related with #37

@cryptofuture
Copy link
Owner

cryptofuture commented Sep 3, 2019

I think it would be better to remove module categories in the config file all together, those are not exactly correct anyway.
I hardly doubt anyone who would be using a module would be loading it without a knowledge about module purpose anyway.
@seonghobae What you think?
Screenshot from 2019-09-03 08-01-50
Also I don't like those ...., I think you text editor creates those, mb just newlines

@cryptofuture
Copy link
Owner

Actually I think if its worth, its worth to only categorize by module name and its *so files

@seonghobae
Copy link
Contributor Author

Yeah, I agree, That just initial version of categorisation. Make descriptor of each .so files will be helpful who want to apply them.

@seonghobae
Copy link
Contributor Author

This project is your project, decide on your way.

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