-
Couldn't load subscription status.
- Fork 9.2k
build.sh accepts version argument #4342
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
Conversation
build.sh with no arguments builds all versions build.sh <version> builds single version build.sh latest builds latest version build.sh src builds from src/oas.md Resolve Google Analytics snippet location relative to md2html.js Update tests to reflect inclusion of snippet Pre-requisite to OAI#4341
ad15ba6 to
9449ce8
Compare
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.
Cool! Only one desired change.
| if [ "$1" = "src" ]; then | ||
| destination="$deploydir/$version.html" | ||
| maintainers="EDITORS.md" | ||
| else | ||
| destination="$deploydir/v$version.html" | ||
| maintainers="$(dirname $specification)/$version-editors.md" | ||
| fi |
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.
Move this up and set deploydir to deploy-preview only for "src".
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.
I moved this as far up the block as I could, but as these values depend on $version, they need to be defined in-loop.
|
One more thing:
|
build.sh with no arguments builds all versions build.sh <version> builds single version build.sh latest builds latest version build.sh src builds from src/oas.md Resolve Google Analytics snippet location relative to md2html.js Update tests to reflect inclusion of snippet Pre-requisite to OAI#4341
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.
Almost there! Just one nit for the src special case on *dev branches.
build.shwith no arguments builds all versionsbuild.sh <version>builds single versionbuild.sh latestbuilds latest versionbuild.sh srcbuilds from src/oas.mdPre-requisite to #4341