-
Notifications
You must be signed in to change notification settings - Fork 6.8k
[MXNET-250] CSS and .htaccess fix #10286
[MXNET-250] CSS and .htaccess fix #10286
Conversation
We need to have the package links in there, but this will be handled with a subsequent PR. |
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.
Some minor comments
@@ -68,6 +68,7 @@ for tag in $tag_list; do | |||
git checkout master -- docs/mxdoc.py | |||
fi | |||
git submodule update || exit 1 | |||
git submodule update --init --recursive |
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.
Why not update line 70?
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 have tried updating it together, but its unstable. I have found this to be better and working consistently
# During a nightly build, these fixes will be patched to all the versions in the asf-site repository including the master folder under versions directory. | ||
# copy <master folder location> <version folder location> | ||
|
||
#SIM: https://sim.amazon.com/issues/AWSDBUX-19727 |
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 don't think we should put public unaccessible link here
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.
Done
python AddPackageLink.py --file_path "$built/versions/master/install/index.html" \ | ||
--current_version "$tag" || exit 1 | ||
fi | ||
#if [ $tag != 'master' ] |
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.
Remove unused code?
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.
Done
* css_and_install_fixes * fixed the broken installation page * copy install index.html fix to 1.1.0 and master versions * fixed install.md * fixed variable definition and install copy logic * added .htaccess and updated .htaccess copy to root folder * copy .htaccess to root folder * updated update_all_version.sh * fixed renaming of the function call * reverted back AddVersion.py from master * removed duplicate master being appended * commented out AddPackageLink.py to fix the master install page issue * removed commented out lines and not required
* css_and_install_fixes * fixed the broken installation page * copy install index.html fix to 1.1.0 and master versions * fixed install.md * fixed variable definition and install copy logic * added .htaccess and updated .htaccess copy to root folder * copy .htaccess to root folder * updated update_all_version.sh * fixed renaming of the function call * reverted back AddVersion.py from master * removed duplicate master being appended * commented out AddPackageLink.py to fix the master install page issue * removed commented out lines and not required
Description
[ ] Copy .htaccess from master to root folder.
[ ] Copy CSS fix (mxnet.css) into all the versions.
[ ] Test out link - http://34.207.78.24/