Skip to content

Conversation

alexrashed
Copy link
Member

The integration with coveralls.io is broken for a while.
After CircleCI's incident in January we rotated all secrets (including the coveralls token).
The token has been updated. With this PR we also upgrade to the latest version to keep up to date.

@alexrashed alexrashed temporarily deployed to localstack-ext-tests February 15, 2023 14:31 — with GitHub Actions Inactive
@github-actions
Copy link

github-actions bot commented Feb 15, 2023

LocalStack integration with Pro

       2 files   -     1         2 suites   - 1   1h 34m 38s ⏱️ + 5m 56s
1 726 tests ±    0  1 366 ✔️  -     2  360 💤 +    2  0 ±0 
2 075 runs   - 365  1 541 ✔️  - 201  534 💤  - 164  0 ±0 

Results for commit 474ee76. ± Comparison against base commit 0448c07.

♻️ This comment has been updated with latest results.

@alexrashed alexrashed temporarily deployed to localstack-ext-tests February 15, 2023 14:45 — with GitHub Actions Inactive
Copy link
Member

@dominikschubert dominikschubert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Didn't even notice that this wasn't working for the last few weeks 😬

@coveralls
Copy link

Coverage Status

Coverage: 84.963% (+0.009%) from 84.954% when pulling 474ee76 on upgrade-coveralls into 0448c07 on master.

@alexrashed alexrashed marked this pull request as ready for review February 15, 2023 15:21
@whummer
Copy link
Member

whummer commented Feb 15, 2023

Thanks for the fix @alexrashed ! Merging the PR, to trigger another build including latest -ext.. ;)

@whummer whummer merged commit 2e16c94 into master Feb 15, 2023
@whummer whummer deleted the upgrade-coveralls branch February 15, 2023 17:11
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.

4 participants