hablapps / doric

Type safety for spark columns
https://www.hablapps.com/doric/
Apache License 2.0
76 stars 11 forks source link

Ci/cache refactor and ci versions update #436

Open eruizalo opened 6 months ago

eruizalo commented 6 months ago

Description

Right now cache action is executing later than it should be, creating tons of caches and not using them. Now we are sharing the cache and avoiding the install action if it is not needed

Related Issue and dependencies

How Has This Been Tested?

github-actions[bot] commented 6 months ago

:octocat: This is an auto-generated comment created by:

Actor Triggering actor Sender
eruizalo
eruizalo
eruizalo
eruizalo
eruizalo
eruizalo
Triggered by: - Workflow name: "CI" at .github/workflows/ci.yml - URL: [https://github.com/hablapps/doric/actions/runs/8283969490](https://github.com/hablapps/doric/actions/runs/8283969490) - on workflow_run:completed

Test summary report 📊

Spark version testing
3.0.0 684 passed, 2 skipped
3.0.1 684 passed, 2 skipped
3.0.2 684 passed, 2 skipped
3.0 684 passed, 2 skipped
3.1.0 712 passed, 2 skipped
3.1.1 712 passed, 2 skipped
3.1.2 712 passed, 2 skipped
3.1 712 passed, 2 skipped
3.2.0 716 passed, 2 skipped
3.2.1 716 passed, 2 skipped
3.2.2 716 passed, 2 skipped
3.2 716 passed, 2 skipped
3.3.0 716 passed, 2 skipped
3.3.1 716 passed, 2 skipped
3.3.2 716 passed, 2 skipped
3.3.3 716 passed, 2 skipped
3.3 716 passed, 2 skipped
3.4.0 716 passed, 2 skipped
3.4.1 716 passed, 2 skipped
3.4 716 passed, 2 skipped
3.5.0 716 passed, 2 skipped
3.5 716 passed, 2 skipped
codecov[bot] commented 6 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 97.70%. Comparing base (fee55fc) to head (af81780).

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/hablapps/doric/pull/436/graphs/tree.svg?width=650&height=150&src=pr&token=N7ZXUXZX1I&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=hablapps)](https://app.codecov.io/gh/hablapps/doric/pull/436?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=hablapps) ```diff @@ Coverage Diff @@ ## main #436 +/- ## ========================================== + Coverage 97.69% 97.70% +0.01% ========================================== Files 62 64 +2 Lines 1255 1259 +4 Branches 37 37 ========================================== + Hits 1226 1230 +4 Misses 29 29 ``` | [Flag](https://app.codecov.io/gh/hablapps/doric/pull/436/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=hablapps) | Coverage Δ | | |---|---|---| | [spark-3.0.x](https://app.codecov.io/gh/hablapps/doric/pull/436/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=hablapps) | `96.72% <ø> (+0.01%)` | :arrow_up: | | [spark-3.1.x](https://app.codecov.io/gh/hablapps/doric/pull/436/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=hablapps) | `97.45% <ø> (+<0.01%)` | :arrow_up: | | [spark-3.2.x](https://app.codecov.io/gh/hablapps/doric/pull/436/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=hablapps) | `97.67% <ø> (+0.01%)` | :arrow_up: | | [spark-3.3.x](https://app.codecov.io/gh/hablapps/doric/pull/436/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=hablapps) | `97.67% <ø> (+0.01%)` | :arrow_up: | | [spark-3.4.x](https://app.codecov.io/gh/hablapps/doric/pull/436/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=hablapps) | `97.67% <ø> (+0.01%)` | :arrow_up: | | [spark-3.5.x](https://app.codecov.io/gh/hablapps/doric/pull/436/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=hablapps) | `97.67% <ø> (+0.01%)` | :arrow_up: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=hablapps#carryforward-flags-in-the-pull-request-comment) to find out more. [see 2 files with indirect coverage changes](https://app.codecov.io/gh/hablapps/doric/pull/436/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=hablapps) ------ [Continue to review full report in Codecov by Sentry](https://app.codecov.io/gh/hablapps/doric/pull/436?dropdown=coverage&src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=hablapps). > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=hablapps) > `Δ = absolute (impact)`, `ø = not affected`, `? = missing data` > Powered by [Codecov](https://app.codecov.io/gh/hablapps/doric/pull/436?dropdown=coverage&src=pr&el=footer&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=hablapps). Last update [fee55fc...af81780](https://app.codecov.io/gh/hablapps/doric/pull/436?dropdown=coverage&src=pr&el=lastupdated&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=hablapps). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=hablapps).