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

Feat/296 replace olafurpg setup scala #320

Merged

Conversation

eruizalo
Copy link
Collaborator

@eruizalo eruizalo commented Jan 23, 2023

Description

Change CI setup actions to get rid of GitHub actions warnings

Related Issue and dependencies

How Has This Been Tested?

  • This pull request contains appropriate tests?:
    • The very CI workflow itself

@eruizalo eruizalo requested a review from a team as a code owner January 23, 2023 15:44
@github-actions github-actions bot added the CI/CD Continuous integration and continuous delivery label Jan 23, 2023
@eruizalo eruizalo enabled auto-merge (squash) January 23, 2023 15:48
@codecov
Copy link

codecov bot commented Jan 23, 2023

Codecov Report

Merging #320 (f68274d) into main (3aa27ad) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #320   +/-   ##
=======================================
  Coverage   97.35%   97.35%           
=======================================
  Files          58       58           
  Lines        1134     1134           
  Branches       17       22    +5     
=======================================
  Hits         1104     1104           
  Misses         30       30           
Flag Coverage Δ
spark-2.4.x 94.87% <ø> (ø)
spark-3.0.x 96.66% <ø> (ø)
spark-3.1.x 97.46% <ø> (ø)
spark-3.2.x 97.46% <ø> (ø)
spark-3.3.x 97.46% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3aa27ad...f68274d. Read the comment docs.

@github-actions
Copy link

github-actions bot commented Jan 23, 2023

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

  • Date : 2023-01-24 08:40:13 +0000 (UTC)
  • Workflow : PR comment
  • Job name : create_test_summary_report
  • Run : 3994398052
  • Commit : f68274d ci: [~] upgrade test-summary action
Actor Triggering actor Sender
eruizalo
eruizalo
eruizalo
eruizalo
eruizalo
eruizalo
Triggered by:

Test summary report 📊

Spark version testing
2.4.1 588 passed, 2 skipped
2.4.2 588 passed, 2 skipped
2.4.3 588 passed, 2 skipped
2.4.4 588 passed, 2 skipped
2.4.5 588 passed, 2 skipped
2.4.6 589 passed, 2 skipped
2.4.7 589 passed, 2 skipped
2.4 589 passed, 2 skipped
3.0.0 621 passed, 2 skipped
3.0.1 621 passed, 2 skipped
3.0.2 621 passed, 2 skipped
3.0 621 passed, 2 skipped
3.1.0 649 passed, 2 skipped
3.1.1 649 passed, 2 skipped
3.1.2 649 passed, 2 skipped
3.1 649 passed, 2 skipped
3.2.0 650 passed, 2 skipped
3.2.1 650 passed, 2 skipped
3.2 650 passed, 2 skipped
3.3.0 650 passed, 2 skipped
3.3 650 passed, 2 skipped

Copy link

@Ernesto-VL Ernesto-VL left a comment

Choose a reason for hiding this comment

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

LGTM

@eruizalo eruizalo merged commit c78276a into hablapps:main Jan 24, 2023
@eruizalo eruizalo deleted the feat/296-replace-olafurpg_setup_scala branch January 24, 2023 08:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI/CD Continuous integration and continuous delivery
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Github actions deprecations [Feature request]: Replace olafurpg/setup-scala action
2 participants