Skip to content
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.

Sign up for GitHub

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

Jump to bottom

fix/integration tests throttling #6171

Merged
merged 14 commits into from
Feb 18, 2025
Merged

fix/integration tests throttling #6171

merged 14 commits into from
Feb 18, 2025

Conversation

baywet
Copy link
Member

@baywet baywet commented Feb 18, 2025

  • ci: removes path ignore from required checks
  • ci: adds token sign in for download operations to reduce likeliness of throttling by GitHub APIs

@baywet baywet self-assigned this Feb 18, 2025
@baywet baywet mentioned this pull request Feb 18, 2025
Merged
@baywet
Revert "ci: adds token sign in for download operations to reduce like…
0d9e969 
…liness of throttling by GitHub APIs"

This reverts commit 07c9f3e.
@baywet
ci: draft download description once for all languages
5a61909 
Signed-off-by: Vincent Biret <[email protected]>
@baywet
chore: refactoring to avoid duplication
6107654 
Signed-off-by: Vincent Biret <[email protected]>
@baywet
ci: fix missing quotes
069ed96 
Signed-off-by: Vincent Biret <[email protected]>
@baywet
ci: adds description caching to idempotency tests
79848a2 
Signed-off-by: Vincent Biret <[email protected]>
@baywet
ci: get a coffee vincent 🤦
229ff0d 
Signed-off-by: Vincent Biret <[email protected]>
@baywet
ci: fix parameter name mismatch
0c7bfe3 
Signed-off-by: Vincent Biret <[email protected]>
@baywet
chore: comment for magic number
b85d03b 
Signed-off-by: Vincent Biret <[email protected]>
@baywet baywet marked this pull request as ready for review February 18, 2025 14:09
@baywet baywet requested a review from a team as a code owner February 18, 2025 14:09
@baywet baywet enabled auto-merge (squash) February 18, 2025 14:10
@baywet
Copy link
Member Author

baywet commented Feb 18, 2025

The brownout might be wrongfully affecting us as well.
actions/toolkit#1952

thewahome
thewahome previously approved these changes Feb 18, 2025
View reviewed changes
@baywet
ci: replaces deprecated action
0dd7251 
Signed-off-by: Vincent Biret <[email protected]>
@baywet
ci: fix missing shell type
849cbe0 
Signed-off-by: Vincent Biret <[email protected]>
@baywet
ci: updates node version
f62dc76 
Signed-off-by: Vincent Biret <[email protected]>
@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed for 'kiota'

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@baywet baywet merged commit b3e3aba into main Feb 18, 2025
283 checks passed
@baywet baywet deleted the fix/integration-tests-throttling branch February 18, 2025 18:55
@thewahome thewahome mentioned this pull request Feb 27, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@andrueastman andrueastman andrueastman approved these changes

@thewahome thewahome thewahome approved these changes

Assignees

@baywet baywet

Labels
None yet
Projects
Kiota
Status: Done ✔️
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@baywet @andrueastman @thewahome