Skip to content
This repository has been archived by the owner on Apr 17, 2023. It is now read-only.

[RAINCATCH 1399] docs fix #195

Merged
merged 6 commits into from Nov 30, 2017
Merged

[RAINCATCH 1399] docs fix #195

merged 6 commits into from Nov 30, 2017

Conversation

paolobueno
Copy link
Contributor

Motivation

Fix API documentation generation script

Description


echo "Rename root folder"
cd $DOCS_LOCATION
mv @raincatcher $VERSION
mv $DOCS_LOCATION/@raincatcher $DOCS_LOCATION/$VERSION
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That was against all bash recommendations! thanks for fixing this.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think a subshell would be okay too (cd $DOCS_LOCATION && mv ...)

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 6b4a4e4 on RAINCATCH-1399-docs-fix into ** on master**.

@wtrocki wtrocki self-requested a review November 30, 2017 16:45
@coveralls
Copy link

Coverage Status

Changes Unknown when pulling b8b3d72 on RAINCATCH-1399-docs-fix into ** on master**.

1 similar comment
@coveralls
Copy link

Coverage Status

Changes Unknown when pulling b8b3d72 on RAINCATCH-1399-docs-fix into ** on master**.

@paolobueno paolobueno merged commit 23ca3b7 into master Nov 30, 2017
@paolobueno paolobueno deleted the RAINCATCH-1399-docs-fix branch November 30, 2017 17:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants