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

ci: remove code owners from .integrated_tests.yaml #3503

Open
wants to merge 21 commits into
base: develop
Choose a base branch
from

Conversation

paveltomin
Copy link
Contributor

to avoid new review requirements popping up when baselines are updated and PR going from approved state into review required

to avoid new review requirements popping up when baselines are updated and PR going from approved state into review required
@paveltomin paveltomin requested a review from CusiniM December 23, 2024 23:12
@paveltomin paveltomin self-assigned this Dec 23, 2024
@paveltomin paveltomin changed the title remove code owners from .integrated_tests.yaml ci: remove code owners from .integrated_tests.yaml Dec 23, 2024
@paveltomin paveltomin added the type: CI Concerns github workflows or generic CI label Dec 23, 2024
Copy link
Collaborator

@CusiniM CusiniM left a comment

Choose a reason for hiding this comment

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

okay, forget my comment in the other PR then. I am fine with this.

@paveltomin paveltomin added ci: run CUDA builds Allows to triggers (costly) CUDA jobs ci: run integrated tests Allows to run the integrated tests in GEOS CI ci: run code coverage enables running of the code coverage CI jobs and removed flag: ready for review labels Feb 6, 2025
Copy link

codecov bot commented Feb 6, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 56.76%. Comparing base (77fe965) to head (42ff2dc).
Report is 1 commits behind head on develop.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #3503   +/-   ##
========================================
  Coverage    56.76%   56.76%           
========================================
  Files         1169     1169           
  Lines       101611   101611           
========================================
  Hits         57677    57677           
  Misses       43934    43934           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@paveltomin paveltomin added flag: requires rebaseline Requires rebaseline branch in integratedTests and removed flag: no rebaseline Does not require rebaseline labels Feb 8, 2025
@paveltomin paveltomin requested a review from cssherman as a code owner February 8, 2025 01:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci: run code coverage enables running of the code coverage CI jobs ci: run CUDA builds Allows to triggers (costly) CUDA jobs ci: run integrated tests Allows to run the integrated tests in GEOS CI flag: requires rebaseline Requires rebaseline branch in integratedTests type: CI Concerns github workflows or generic CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants