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.

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

Adding codecov CI workflow. #2

Closed
wants to merge 19 commits into from
Closed

Adding codecov CI workflow. #2

wants to merge 19 commits into from

Conversation

mattpolzin
Copy link
Owner

@mattpolzin mattpolzin commented Jan 18, 2021

A draft to test code coverage setup.

@mattpolzin
Copy link
Owner Author

mattpolzin commented Jan 19, 2021

The current code coverage percentage is 18.43% (minimum allowed: 20%).

-Overall Coverage: 20.30%
+Overall Coverage: 18.43%
 
 File                               Coverage
 ---------------------------------- --------
 ButtonStyle.swift                  35.14%  
 CGStubs.swift                      0.00%   
 ClipEffect.swift                   0.00%   
-Color.swift                        19.34%  
+Color.swift                        0.00%   
 ColorScheme.swift                  0.00%   
 DisclosureGroup.swift              0.00%   
 Divider.swift                      0.00%   
                                            
 =-=-=-=-=-=-=-=-=                          
                                            
-ColorTests.swift                   88.89%  
 ReconcilerTests.swift              93.24%   ```

@mattpolzin mattpolzin closed this Feb 4, 2021
MaxDesiatov pushed a commit to TokamakUI/Tokamak that referenced this pull request Feb 5, 2021
Closes #350.

Adds simple code coverage analysis. Until the GitHub token is set up in this repo, you can see the results including a comment on the PR here: mattpolzin#2

* Adding codecov CI workflow.

* doh. forgot to skip build.

* drop unneeded llvm env var from build invocation. make comment always run.

* try a way to update a previous comment

* try to get comments to run on failure.

* use bug fix to codecov action.

* fix comment search string and drop min coverage

* use replace on comment updates

* attempt at diffing coverage from main branch

* comment out tests to affect test coverage

* try method for multiline outputs

* do i need to put it in its own script file

* right, switch coverage gen order to have branch checked out second.

* attempt to get code quotation around diff output.

* switch to git diff

* add note about the new script.

* try echoing a warning.

* post warning on success and error on failure.

* uncomment tests
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.

1 participant