Skip to content

add modules in nginx.conf - #39

Open
seonghobae wants to merge 5 commits into
cryptofuture:masterfrom
seonghobae:typos
Open

add modules in nginx.conf#39
seonghobae wants to merge 5 commits into
cryptofuture:masterfrom
seonghobae:typos

Conversation

@seonghobae

Copy link
Copy Markdown

related with #37

@cryptofuture

cryptofuture commented Sep 3, 2019

Copy link
Copy Markdown
Owner

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
Copy Markdown
Owner

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

@seonghobae

Copy link
Copy Markdown
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
Copy Markdown
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.

2 participants