Skip to content

Conversation

@LeiWang1999
Copy link
Contributor

This pull request includes a variety of changes across multiple files, primarily focusing on improving the installation process, refining testing procedures, and enhancing error handling. The changes also include the introduction of a new continuous integration workflow and modifications to the codebase for better compatibility with different branches.

Here are the most significant changes:

Continuous Integration:

  • .github/workflows/ci.yml: Introduced a new continuous integration workflow that triggers on push and pull request events. It sets up a Python environment, installs dependencies, and runs tests on a self-hosted runner.

Installation Improvements:

  • install.sh, maint/scripts/installation.sh: Updated the scripts to install LLVM by fetching the appropriate version based on the Ubuntu version. The scripts now also handle different architectures (x86_64 and aarch64). [1] [2]

Compatibility Enhancements:

  • format.sh: Updated the script to determine the base branch dynamically, which allows it to work with different branch names (e.g., origin/main or main). This change was applied to the format_changed(), spell_check_changed(), and lint_changed() functions. [1] [2] [3]

Other Changes:

  • setup.py: Several changes were made to this file, including updating the package directory, removing the creation of a softlink for bitblas, and adjusting the versioning mechanism. [1] [2] [3] [4] [5]
  • requirements-dev.txt: Added wheel and setuptools to the list of development dependencies.

CI will automatically run the format.sh once a pull request be created.

To do test locally, please run pytest in testing/python directory.

LeiWang1999 and others added 30 commits July 1, 2024 16:57
@LeiWang1999 LeiWang1999 merged commit 1b7e52d into microsoft:main Jul 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant