-
Notifications
You must be signed in to change notification settings - Fork 6.8k
[website] Fix broken website for master version #19945
Conversation
Hey @waytrue17 , Thanks for submitting the PR
CI supported jobs: [unix-gpu, unix-cpu, windows-cpu, edge, clang, website, centos-gpu, centos-cpu, sanity, miscellaneous, windows-gpu] Note: |
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.
LGTM!
Thank you. |
@mxnet-bot run ci [windows-cpu] |
Jenkins CI successfully triggered : [windows-cpu] |
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.
LGTM, thanks Wei!
@waytrue17 any ideas how to update our CI to fail if a PR introduces problems similar to #19743? |
could add a snapshot check in CI? |
* fix website * fix header * fix index * remove locall test * fix main.scss Co-authored-by: Wei Chu <weichu@amazon.com>
* fix website * fix header * fix index * remove locall test * fix main.scss Co-authored-by: Wei Chu <weichu@amazon.com>
* fix website * fix header * fix index * remove locall test * fix main.scss Co-authored-by: Wei Chu <weichu@amazon.com>
* fix website * fix header * fix index * remove locall test * fix main.scss Co-authored-by: Wei Chu <weichu@amazon.com> Co-authored-by: waytrue17 <52505574+waytrue17@users.noreply.github.com> Co-authored-by: Wei Chu <weichu@amazon.com>
…from master (#20785) * [v1.9.x] Fix CD for pypi wheel version (#20782) * Change env variable used to detect whether it's a release build or not. RELEASE_BUILD is already setup in the CD pipeline. * Refine how variable is compared to trigger release builds. * Check for 'true' exclusively, to prevent unwanted release builds. * Make sure we pass any environment variables on the build.py command line to the docker run command. * [v1.9.x] Fix aarch64 cd pipeline (#20783) * Add wheel to installed pip packages on aarch64, so we can build the wheel in the CD pipeline. * Better support for aarch64 docker container cache. * Only import machine() function from platform module, as platform variable is already widely used. * Only build containers with aarch64 in name on aarch64. * Make all arch defaults to local machine type. * Retry getting test data to prevent CI tests from failing - increase connect timeout from 10 to 60sec, increase retry delay from 0 (setting duplicated) to 30, increase retry attempts to 5. * Update website for v1.9.x branch. (#20786) * [website] Fix broken website for master version (#19945) * fix website * fix header * fix index * remove locall test * fix main.scss Co-authored-by: Wei Chu <weichu@amazon.com> Co-authored-by: waytrue17 <52505574+waytrue17@users.noreply.github.com> Co-authored-by: Wei Chu <weichu@amazon.com>
Description
Fix broken website for master version described in #19943. The issue is due to the format of license header introduced by #19743, which blocks the pages from building by Jekyll:
This PR attempts to fix the broken website by rearranging the license format.
Checklist
Essentials
Changes
Comments