-
Notifications
You must be signed in to change notification settings - Fork 75
feat: add pr update webhook #466
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
base: main
Are you sure you want to change the base?
Conversation
SaboniAmine
left a comment
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! Do we need to activate the Github Action via sponsoring first ?
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.
Comment @cursor review or bugbot run to trigger another review on this PR
…pruna_internal - Added new package sources for `fast-hadamard-transform`, `flute-kernel`, `pit`, `pruna`, `pruna-engine`, `pruna-launch-party`, and `pruna-pro`, ensuring they are sourced from `pruna_internal` by default. - Maintained existing source configurations for `gptqmodel` to handle Apple Silicon Darwin compatibility.
- Removed package sources for `fast-hadamard-transform`, `flute-kernel`, `pit`, `pruna`, `pruna-engine`, `pruna-launch-party`, and `pruna-pro` from `pyproject.toml`, streamlining the configuration. - Retained the existing source configuration for `stable-fast` and `gptqmodel` to ensure compatibility with `pruna_internal`.
- Cleaned up `pyproject.toml` by removing unnecessary empty lines and ensuring a more streamlined configuration for package sources. - Maintained existing configurations for `stable-fast` and `gptqmodel` to preserve functionality.
- Adjusted the `default` priority for the `pruna_internal` index to false, ensuring it has the highest priority for package sourcing. - Set the `default` priority for the `pypi` index to true, making it the lowest priority for package sourcing. - These changes streamline package resolution and improve dependency management.
- Reverted the `default` priority for the `pruna_internal` index to true, making it the lowest priority for package sourcing. - Adjusted the `default` priority for the `pypi` index to false, ensuring it has the highest priority for package sourcing. - These changes enhance package resolution and dependency management.
- Added "N813" to the ignore list for Ruff linting. - Specified linting rules for "src/pruna/algorithms/base/tags.py" to ignore "E501". - Introduced new rules for type checking with Ty, allowing for a smoother transition from mypy. - Updated package versions, including changing "datasets" to ">=0.34" and adding "numpydoc" and "imageio-ffmpeg" to dependencies. - Bumped project version from 0.2.8 to 0.3.0 for the upcoming release.
e917b91 to
e1067f4
Compare
|
@SaboniAmine the Slack notification integration requires sponsoring. |
minettekaum
left a comment
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 :D
Description
Add and overview of PR progress to Slack channel
Related Issue
Fixes NA
Type of Change
How Has This Been Tested?
Checklist
Additional Notes