Skip to content

Conversation

@alamb
Copy link
Owner

@alamb alamb commented Jun 12, 2021

(testing PR against alamb/arrow-rs)

Rationale:

Locate PRs that affect various parts of the codebase to make it easier to find PRs in need of review

Changes:

  1. Add labeler action, using standard github action (https://github.com/actions/labeler)

Notes

You can see this working on https://github.com/alamb/arrow-rs (TODO link)

@alamb alamb merged commit 0cf6ef0 into master Jun 12, 2021
This was referenced Jun 12, 2021
@alamb alamb deleted the alamb/labeler branch June 13, 2021 10:55
alamb pushed a commit that referenced this pull request Mar 21, 2023
alamb pushed a commit that referenced this pull request Aug 6, 2024
separate tests that require arrow into a separate module
alamb pushed a commit that referenced this pull request Nov 25, 2024
* Implement bulk_delete_request for Azure

* Fix lint and add Azurite bug workaround

* Special 404 error case

* Clippy fix

* Make number of expected headers more conservative and better document invariants

* Use multer for multipart parsing

* Fix clippy

* Fix clippy #2

* Reuse part response buffer

* Make multer conditional to azure feature

* One more HeaderValue::from_static

* Add tests for bulk delete request building and response parsing

* Switch back to manual parsing to avoid multer dependency, other PR suggestions

* Fixes lint
alamb pushed a commit that referenced this pull request Mar 20, 2025
* Implement bulk_delete_request for Azure

* Fix lint and add Azurite bug workaround

* Special 404 error case

* Clippy fix

* Make number of expected headers more conservative and better document invariants

* Use multer for multipart parsing

* Fix clippy

* Fix clippy #2

* Reuse part response buffer

* Make multer conditional to azure feature

* One more HeaderValue::from_static

* Add tests for bulk delete request building and response parsing

* Switch back to manual parsing to avoid multer dependency, other PR suggestions

* Fixes lint
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