Skip to content

support to add relative path dynamic-module in configure#343

Closed
catatsuy wants to merge 2 commits intoopenresty:masterfrom
catatsuy:feature/available_add_dynamic_module_rel_dir
Closed

support to add relative path dynamic-module in configure#343
catatsuy wants to merge 2 commits intoopenresty:masterfrom
catatsuy:feature/available_add_dynamic_module_rel_dir

Conversation

@catatsuy
Copy link
Contributor

I hereby granted the copyright of the changes in this pull request
to the authors of this openresty project.

@agentzh
Copy link
Member

agentzh commented Feb 11, 2018

@catatsuy Looking good to me. Mind to add a test case to the existing test suite to cover this new feature? Thanks!

@catatsuy
Copy link
Contributor Author

@agentzh I added a test. Please check.

@TiburonzinGT
Copy link

@agentzh Hello, I would like to translate to Spanish but I do not know how. I would like to do it by means of pull request :)

@agentzh
Copy link
Member

agentzh commented Feb 25, 2018

@shadowz10 Please do not hijack unrelated PRs or issues. Thank you.

@agentzh
Copy link
Member

agentzh commented Feb 25, 2018

@catatsuy Sorry for the delay on my side. Just merged it with fixes for the test failures.

@agentzh agentzh closed this Feb 25, 2018
@agentzh
Copy link
Member

agentzh commented Feb 25, 2018

Thanks!

@catatsuy
Copy link
Contributor Author

Thank you!

@catatsuy catatsuy deleted the feature/available_add_dynamic_module_rel_dir branch February 26, 2018 01:01
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.

3 participants