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

Revert the changes to codecov.yml #482

Merged
merged 1 commit into from
Dec 4, 2023
Merged

Conversation

DilumAluthge
Copy link
Member

Targets #477

@DilumAluthge DilumAluthge merged commit 5487c99 into dpa/merge-queue Dec 4, 2023
7 checks passed
@DilumAluthge DilumAluthge deleted the dpa/codecov branch December 4, 2023 21:01
github-merge-queue bot pushed a commit that referenced this pull request Jan 8, 2024
…, and add some more unit tests (#477)

* Run the integration tests on the `merge_group` event

* Run the unit tests on the `merge_group` event

* Delete `bors.toml` (the Bors config file)

* CompatHelper's own Project.toml: add compat entries for our own test deps (#478)

* Update .github/workflows/ci_integration.yml

Co-authored-by: mattBrzezinski <[email protected]>

* Update ci_integration.yml

* Upload code coverage from the unit tests

* Integration tests: immediately error if this is a PR build

* Mark the integration tests as "skipped" on PR builds (#479)

* Simplify the job name of the integration tests job

This is necessary because the `${{ matrix... }}` interpolations are not evaluated for skipped jobs

* Do some debugging

* Revert the debugging commit

* Don't use a matrix for the integration test job

* Do some debugging

* Revert the debugging commit

* Delete some commented-out lines

* Fix a comment

* add codecov threshold (#480)

* add codecov threshold

to help with #477

* Update codecov.yml

* fix indentation

* Update codecov.yml

* Update codecov.yml

* Apply suggestions from code review

Co-authored-by: Dilum Aluthge <[email protected]>

* Update codecov.yml

* try to make two settings exclusive

* Refactor some code, and increase the unit test code coverage for `src/main.jl` (#481)

* Revert the changes to `codecov.yml` (#482)

* Lower Codecov threshold to 99% because of GitHub Merge Queue limitations (#484)

---------

Co-authored-by: Eric Hanson <[email protected]>
Co-authored-by: mattBrzezinski <[email protected]>
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