fixed timeout methods not being feature gated #519
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Thank you for your PR! However, I think what happened is that we forgot to remove the feature gate everywhere, so I'd prefer to remove all mentions of |
CommanderStorm
added a commit
to TUM-Dev/NavigaTUM
that referenced
this pull request
Jan 15, 2024
…tion error This is an interim fix until XAMPPRocky/octocrab#519 is merged
|
I have refactored this PR as such. |
CommanderStorm
added a commit
to TUM-Dev/NavigaTUM
that referenced
this pull request
Jan 15, 2024
* Update Rust crate octocrab to 0.33.0 * fixed the timeout feature not being enabled => thus causing a compilation error This is an interim fix until XAMPPRocky/octocrab#519 is merged --------- Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: Frank Elsinga <frank@elsinga.de>
|
Ah okay, sorry I guess I didn't understand that this would then require tokio, sorry for the extra work but if you revert to your original PR, we can merge that 🙂 |
This reverts commit 6024850.
No worries ^^ Have reverted the commit |
|
Thank you for your PR, and congrats on your first contribution! 🎉 |
Merged
dmgorsky
pushed a commit
to dmgorsky/octocrab
that referenced
this pull request
Mar 6, 2024
* fixed timeout methods not being feature gated * removed the `timeout` feature gate as requested * Revert "removed the `timeout` feature gate as requested" This reverts commit 6024850.
XAMPPRocky
pushed a commit
that referenced
this pull request
Mar 7, 2024
* added /repos/{owner}/{repo}/check-suites
* added /repos/{owner}/{repo}/check-suites/preferences
* added /repos/{owner}/{repo}/check-suites/{check_suite_id}
* added rerequest check [run/suite]
* added 'list check run annotations'
* added "List check suites for a Git reference"
* added query parameters
* fixed `cargo fmt --all`
* fixed timeout methods not being feature gated (#519)
* fixed timeout methods not being feature gated
* removed the `timeout` feature gate as requested
* Revert "removed the `timeout` feature gate as requested"
This reverts commit 6024850.
* chore: release (#520)
* Added a test for `repos().is_collaborator()`. The 204 case is good! (#553)
* Resolving issue with deleting reactions from issues. (#555)
* Resolving issue with deleting reactions from issues.
Issue: manchicken#182
* Reverting changes to CHANGELOG.md since that's automated.
* chore: release (#554)
Signed-off-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
* Tests for locking and unlocking issue comments (#557)
* Handle redirects in ReleasesHandler::stream_asset(). (#556)
* chore: release (#558)
* Added tests for `repos().merges()` (#560)
* Don't send Authorization headers to third-party services. (#562)
* chore: release (#561)
* Upgrade http/hyper 1.0 (#564)
* chore: release (#565)
* Fixed `issues().delete_comment_reaction()` (#566)
- Added a test for 204 and 500 cases
- Fixed broken functionality in the 204 case.
* Adding test for `issues().delete_comment()` (#569)
* Added tests to prove `repos().secrets().delete_secret()` (#570)
- Added test for 204 case
- Added test for 500 case
* chore: release (#567)
Signed-off-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
* Added a test (#577)
- Update case of `repos().secrets().create_or_update_secret()` test added
* Added some more tests (#578)
- `repos().issues().check_assignee()`
- Covers 204 case
- Covers 404 case
- Covers 500 case
* Fixing the test that `wiremock@v0.6.0` breaks. (#579)
- Updated wiremock to 0.6.0
- Fixed the one test broken by the update
* Fixing `snafu` version update (#580)
* Adding test for actions().actions_delete_workflow_run_logs_test() (#584)
* Added more tests, for `repos().delete()` (#585)
* Fixed the `projects().delete_project().send() function (#587)
* Adding tests for `actions().remove_selected_repo_from_org_secret()` (#586)
* Add Public and Release events (#589)
* Update base64 requirement from 0.21.0 to 0.22.0 (#592)
Updates the requirements on [base64](https://github.com/marshallpierce/rust-base64) to permit the latest version.
- [Changelog](https://github.com/marshallpierce/rust-base64/blob/master/RELEASE-NOTES.md)
- [Commits](marshallpierce/rust-base64@v0.21.2...v0.22.0)
---
updated-dependencies:
- dependency-name: base64
dependency-type: direct:production
...
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* chore: release (#581)
Signed-off-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
* Adding tests to `orgs().secrets().create_or_update_secret()` (#590)
- Added 204 case
* Add delete_ref in RepoHandler (#573)
Fixes: #572
* chore: release (#593)
Signed-off-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
* rebased to actual; fixed Response<Body>
* added /repos/{owner}/{repo}/check-suites
* added /repos/{owner}/{repo}/check-suites/preferences
* added /repos/{owner}/{repo}/check-suites/{check_suite_id}
* added rerequest check [run/suite]
* added 'list check run annotations'
* added "List check suites for a Git reference"
* added query parameters
* fixed `cargo fmt --all`
* rebased to actual; fixed Response<Body>
---------
Signed-off-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Frank Elsinga <frank@elsinga.de>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Mike Stemle <hello@mikestemle.com>
Co-authored-by: Kor Nielsen <korn@xdas.com>
Co-authored-by: Kor Nielsen <kor@google.com>
Co-authored-by: Toru Ogawa <Hakuyume@users.noreply.github.com>
Co-authored-by: Evan Graham <me@luakt.net>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Tobias Nießen <tniessen@tnie.de>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Note
For those affected: Add the
timeout-feature to get around this bugIn #498 a few methods were introduced which reference fields which are feature-gated.
causing
This PR applies the same feature gate for these methods (reverted f1c94c1)
Cheers and thanks for providing such an usefull crate ^^