-
Notifications
You must be signed in to change notification settings - Fork 3
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
ALFREDAPI-552 update swagger spec to be oa2 compliant #222
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hard to give feedback. If this runs in the appropriate tools, then it is good.
|
||
tasks.register("buildWebsiteScript", Exec) { | ||
setGroup "documentation" | ||
commandLine 'chmod', '+x', './build-website.sh' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we can commit +x with Git, no need to execute it every time
put from: file("${layout.buildDirectory.get()}/website-alfred-api_${today}.tar.gz"), into: "${remoteHomeDir}/docs.xenit.eu/" | ||
execute "mv ${remoteHomeDir}/docs.xenit.eu/alfred-api ${remoteHomeDir}/docs.xenit.eu/alfred-api_${today}_back" | ||
execute "mkdir -p ${remoteHomeDir}/docs.xenit.eu/back/" | ||
execute "tar vxzf ${remoteHomeDir}/docs.xenit.eu/website-alfred-api_${today}.tar.gz -C ${remoteHomeDir}/docs.xenit.eu/back/" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the order might be a bit confusing, would be best to first complete the backup, then deploy the new docs?
This has already been done twice without being committed to the main repository. We should make this a permanent change
Fixes https://xenitsupport.jira.com/browse/ALFREDAPI-552
Breaking changes include adding, changing or removing endpoints and/or JSON objects used in requests and responses.
@ExceptionHandler
in the webscript classes?this.
prefix avoided?See README.md for full explanation.