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

Compile swagger #9

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

Conversation

amnonh
Copy link

@amnonh amnonh commented Oct 23, 2014

This pull request is part of the changes need in the move from static files in the swagger-ui to compiled files in the swagger ui.

It removes the static files and modify the template, so that the end result will be the same.

Note that the swagger-ui will continue to run properly the patch series in the osv repository "Changing the Swagger-UI to compile its files" need to be committed.

When compiling the Swagger-UI the dist directory should not be included in the
repository, instead it and all of its content will be created during the
compilation.

Signed-off-by: Amnon Heiman <[email protected]>
This adaptations insure that the generated files will be equivelent to the
files that were used before moving to compiled files.

Signed-off-by: Amnon Heiman <[email protected]>
@tgrabiec
Copy link
Member

You seem to remove a lot of files. Where these files will come from now?

@amnonh
Copy link
Author

amnonh commented Oct 23, 2014

The dist directory is created during compilation and should be removed in
that case.

On Thu, Oct 23, 2014 at 12:25 PM, Tomasz Grabiec [email protected]
wrote:

You seem to remove a lot of files. Where these files will come from now?


Reply to this email directly or view it on GitHub
#9 (comment)
.

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