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

chore(deps): Update codecov-shared dependency #194

Merged
merged 2 commits into from
Nov 30, 2023

Conversation

FraBle
Copy link
Contributor

@FraBle FraBle commented Nov 29, 2023

Purpose/Motivation

Update shared, which includes a fix for the install schema validation that previously errored when specifying admins, which is required for self-hosting codecov.

Links to relevant tickets

N/A

What does this PR do?

Upgrades shared to the latest Git hash, which includes the fix PR.

Notes to Reviewer

Anything to note to the team? Any tips on how to review, or where to start?

Legal Boilerplate

Look, I get it. The entity doing business as "Sentry" was incorporated in the State of Delaware in 2015 as Functional Software, Inc. In 2022 this entity acquired Codecov and as result Sentry is going to need some rights from me in order to utilize my contributions in this PR. So here's the deal: I retain all rights, title and interest in and to my contributions, and by keeping this boilerplate intact I confirm that Sentry can use, modify, copy, and redistribute my contributions, under Sentry's choice of terms.

@FraBle FraBle changed the title Update codecov-shared dependency chore(deps): Update codecov-shared dependency Nov 29, 2023
requirements.txt Outdated
@@ -16,8 +16,6 @@ anyio==3.6.1
# via
# httpcore
# openai
async-timeout==4.0.2
Copy link
Contributor

Choose a reason for hiding this comment

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

Why did it remove these things?... maybe it's a python 3.12 thing? That is fairly new.

When I run locally it doesn't remove anything. Could you put them back?...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sure, I just pushed a commit reversing the removal.

@giovanni-guidini giovanni-guidini merged commit 8508cc5 into codecov:main Nov 30, 2023
12 checks passed
scott-codecov added a commit that referenced this pull request Dec 6, 2023
* main:
  Only trigger AI PR review if pull is open
  Add log line when triggering AI PR review task
  fix: use original PR base to compute patch coverage (#199)
  Prepare release 23.12.4
  Update workflows
  chore(deps): Update codecov-shared dependency (#194)
  Prep the terrain for reports with label compression. (#188)
  allow staging deploy when pushing to staging (#192)
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.

2 participants