Skip to content

Add CodeCov configuration, enable test analytics (#3022) #3023

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 7 commits into from
Oct 23, 2024
Merged

Add CodeCov configuration, enable test analytics (#3022) #3023

merged 7 commits into from
Oct 23, 2024

Conversation

@tishun
Copy link
Collaborator

@tishun tishun commented Oct 22, 2024 •
edited
Loading

Closes #3022

Make sure that:

  • You have read the contribution guidelines.
  • You have created a feature request first to discuss your contribution intent. Please reference the feature request ticket number in the pull request.
  • You applied code formatting rules using the mvn formatter:format target. Don’t submit any formatting related changes.
  • You submit test cases (unit or integration tests) that back your changes.

@tishun tishun changed the title Add CodeCov configuration, enable test analytics Add CodeCov configuration, enable test analytics (#3022) Oct 22, 2024
ggivo
ggivo reviewed Oct 22, 2024
View reviewed changes
Copy link
Contributor

@ggivo ggivo left a comment

Choose a reason for hiding this comment

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

LGTM

@tishun tishun added the type: task A general task label Oct 23, 2024
@tishun tishun merged commit 29afe13 into redis:main Oct 23, 2024
5 checks passed
@tishun tishun deleted the topic/tishun/issue-3022 branch October 23, 2024 05:43
thachlp pushed a commit to thachlp/lettuce that referenced this pull request Dec 31, 2024
@tishun @thachlp
Add CodeCov configuration, enable test analytics (redis#3022) (redis#…
322432d 
…3023)

* Add CodeCov configuration, enable test analytics

* Issue in argument

* ... and the right filename

* ... perhaps by specifying exactly where the file is?

* Polishing

* Check why configuration is not working

* Final version
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ggivo ggivo ggivo left review comments

Assignees

No one assigned

Labels

type: task A general task

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Report test analytics in CodeCov

2 participants

@tishun @ggivo