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

ci(MM-63199) temp code coverage tracking #8669

Closed
wants to merge 63 commits into from
Closed

ci(MM-63199) temp code coverage tracking #8669

wants to merge 63 commits into from

Conversation

rahimrahman
Copy link
Contributor

Summary

Ticket Link

Checklist

  • Added or updated unit tests (required for all new features)
  • Has UI changes
  • Includes text changes and localization file updates
  • Have tested against the 5 core themes to ensure consistency between them.
  • Have run E2E tests by adding label E2E iOS tests for PR.

Device Information

This PR was tested on:

Screenshots

Release Note

@rahimrahman
ci(MM-63199): code coverage tracking
aa4693b
@rahimrahman
try to download existing coverage file
05847ce
@rahimrahman
read coverage
ece130b
@rahimrahman
add token
137bee6
@rahimrahman
use github.token instead
88fdcc5
@rahimrahman
passing github token
22b2320
@rahimrahman
github_token passing from workflow
12013da
@rahimrahman
remove download
6f4bc8e
@rahimrahman
re-add download
a805c7f
@rahimrahman
wrong param
9040a95
@rahimrahman
try download all artifacts
594bf0d
@rahimrahman
add run-id so to retrieve with download later
c67a2b1
@rahimrahman
remove read coverage temp
2a77069
@rahimrahman
use run-id to download
0f02944
@rahimrahman
put files into current-coverage
74ed15f
@rahimrahman
using last run id
0a2e7f4
@rahimrahman
temporary comment
362b3e0
@rahimrahman
can retrieve last run id?
4392803
@rahimrahman
remove hard-coding
680cf03
@rahimrahman
echo into github_env vs export
1f7237c
@rahimrahman
comparing new and old
bda32c8
@rahimrahman
comparison improvement
deeb5a3
@rahimrahman
post to github
81d3396
@rahimrahman
fix coverage text
efa86a6
@rahimrahman
refactor to main from current-coverage
a211c78
@rahimrahman
formatting changes
80c5ef0
@rahimrahman
fix missing content
e100fe0
@rahimrahman
small tweaking
f8ee286
@rahimrahman
showing the Warning to make sure
68203a8
@rahimrahman
formatting
f6a7678
@rahimrahman
only run certain things on main
2319e3c
@rahimrahman
trying cache-hit
86e573c
@rahimrahman
real trying cache-hit
2375276
@rahimrahman
testing to make sure cache-run-id runs
55f1c08
@rahimrahman
save-always true
c2ef265
@rahimrahman
save-always deprecated
6a7fa05
@rahimrahman
let's try different strategy
e2f5d19
@rahimrahman
add key
3fe2f16
@rahimrahman
restore-key adding a -
9e2631b
@rahimrahman
only perform on main
5d6a287
@rahimrahman
only run on main or if its a PR
22fc24e
@rahimrahman
coverage_threshold
aecc5b6
@rahimrahman
remove comments
47992ea
@rahimrahman
add total
a38547b
@rahimrahman
removing unneeded comments
fb1088b
@rahimrahman
calculate total
199d8f6
@rahimrahman
run test in release-* only
5095eae
@rahimrahman
making sure that only PR will run
957bd6c
@rahimrahman
only do more steps if upload-coverage successful
4ad4846
@rahimrahman
trying thollander/actions-comment-pull-request
2398d58
@rahimrahman
using diff way to comment.
100e2ef
@rahimrahman
comment on how things work
893ea66
@rahimrahman
testing to trigger warning and see if comment is updated vs new comment
62bb5a2
@rahimrahman
omit echo messages
90a801c
@rahimrahman
see if giving github token would work.
3732bd2
@rahimrahman
wrong use of param
6e84d9d
@rahimrahman
try without github token
f1cd18e
@rahimrahman
Copy link
Contributor Author

Not able to get this while the PR is running. Can only work when the branch is coming from a default branch. As an example, default.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
release-note
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@rahimrahman @mm-cloud-bot