Skip to content

S3 coverage comparison#3

Draft
Satevg wants to merge 1 commit intomasterfrom
s3-coverage-comparison
Draft

S3 coverage comparison#3
Satevg wants to merge 1 commit intomasterfrom
s3-coverage-comparison

Conversation

@Satevg
Copy link

@Satevg Satevg commented Oct 13, 2023

⚠️ DO NOT REMOVE THIS BRANCH ⚠️

https://aquicore.atlassian.net/browse/AQ-15182

Frontend repository uses this branch to calculate coverage with remote S3 file as a base.

- name: Test Coverage
id: testCoverage
uses: aquicore/Jest-Coverage-Diff@s3-coverage-comparison

⚠️ DO NOT MERGE THIS BRANCH INTO MASTER ⚠️

This might break services pipelines (except frontend) that rely on aquicore/Jest-Coverage-Diff

@Satevg Satevg marked this pull request as ready for review October 17, 2023 11:41
src/main.ts Outdated

if (!diffChecker.checkIfTestCoverageFallsBelowDelta(delta)) {
const gif = await getGiphyGifForTag('happy dog')
const gif = await getGiphyGifForTag('happy cat')
Copy link
Author

Choose a reason for hiding this comment

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

🐈

@Satevg Satevg marked this pull request as draft October 17, 2023 12:03
@Satevg
Copy link
Author

Satevg commented Oct 17, 2023

Marked as draft, so that it's not possible to accidentally merge this into master

@Satevg Satevg force-pushed the s3-coverage-comparison branch from 225fe23 to 7c23e2d Compare October 18, 2023 10:12
@Satevg Satevg requested a review from rself-ig October 18, 2023 10:12
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.

2 participants