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

Add daily progress principle to PROCESS.md #3098

Merged
merged 5 commits into from
Jul 24, 2023

Conversation

rumyantseva
Copy link
Contributor

@rumyantseva rumyantseva commented Jul 21, 2023

Description

Please read and share your feedback. This is an expectation we have but it doesn't seem to be really visible.

Readiness checklist

  • I added/updated unit tests (and they pass).
  • I added/updated integration/compatibility tests (and they pass).
  • I added/updated comments and checked rendering.
  • I made spot refactorings.
  • I updated user documentation.
  • I ran task all, and it passed.
  • I ensured that PR title is good enough for the changelog.
  • (for maintainers only) I set Reviewers (@FerretDB/core), Labels, Project and project's Sprint fields.
  • I marked all done items in this checklist.

@rumyantseva rumyantseva added project Project maintenance do not merge PRs that should not be merged labels Jul 21, 2023
@rumyantseva rumyantseva self-assigned this Jul 21, 2023
@rumyantseva rumyantseva marked this pull request as ready for review July 21, 2023 12:12
@rumyantseva rumyantseva enabled auto-merge (squash) July 21, 2023 12:12
@codecov
Copy link

codecov bot commented Jul 21, 2023

Codecov Report

Merging #3098 (cfdc99b) into main (1924913) will decrease coverage by 17.37%.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##             main    #3098       +/-   ##
===========================================
- Coverage   72.78%   55.41%   -17.37%     
===========================================
  Files         390      390               
  Lines       21330    21330               
===========================================
- Hits        15526    11821     -3705     
- Misses       4865     8782     +3917     
+ Partials      939      727      -212     

see 93 files with indirect coverage changes

Flag Coverage Δ
integration 46.72% <ø> (-22.37%) ⬇️
mongodb 5.53% <ø> (ø)
pg ?
shard-1 36.55% <ø> (-19.58%) ⬇️
shard-2 36.05% <ø> (-18.12%) ⬇️
shard-3 38.13% <ø> (ø)
sqlite 46.63% <ø> (+0.02%) ⬆️
unit 24.29% <ø> (+0.04%) ⬆️

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

noisersup
noisersup previously approved these changes Jul 21, 2023
b1ron
b1ron previously approved these changes Jul 21, 2023
chilagrow
chilagrow previously approved these changes Jul 24, 2023
Copy link
Member

@chilagrow chilagrow left a comment

Choose a reason for hiding this comment

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

👍

@AlekSi AlekSi added this to the Next milestone Jul 24, 2023
@AlekSi AlekSi added the not ready Issues that are not ready to be worked on; PRs that should skip CI label Jul 24, 2023
.github/PROCESS.md Outdated Show resolved Hide resolved
@rumyantseva rumyantseva dismissed stale reviews from chilagrow, b1ron, and noisersup via 782b531 July 24, 2023 07:46
@rumyantseva rumyantseva requested a review from b1ron July 24, 2023 07:47
@AlekSi AlekSi removed not ready Issues that are not ready to be worked on; PRs that should skip CI do not merge PRs that should not be merged labels Jul 24, 2023
@rumyantseva rumyantseva merged commit 19e1cdb into FerretDB:main Jul 24, 2023
@rumyantseva rumyantseva deleted the daily-progress branch July 24, 2023 18:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
project Project maintenance
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

6 participants