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 a YAML formatter #2485

Merged
merged 9 commits into from
Apr 26, 2023
Merged

Add a YAML formatter #2485

merged 9 commits into from
Apr 26, 2023

Conversation

wqhhust
Copy link
Contributor

@wqhhust wqhhust commented Apr 20, 2023

Description

Closes #1301 .

Readiness checklist

  • I added/updated unit tests.
  • I added/updated integration/compatibility tests.
  • 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.

@wqhhust wqhhust requested a review from AlekSi as a code owner April 20, 2023 01:21
@CLAassistant
Copy link

CLAassistant commented Apr 20, 2023

CLA assistant check
All committers have signed the CLA.

@wqhhust wqhhust mentioned this pull request Apr 20, 2023
@AlekSi AlekSi requested a review from rumyantseva April 20, 2023 08:46
@rumyantseva rumyantseva changed the title Closes #1301 Add a YAML formatter Apr 21, 2023
@rumyantseva rumyantseva added the code/chore Code maintenance improvements label Apr 21, 2023
Copy link
Contributor

@rumyantseva rumyantseva left a comment

Choose a reason for hiding this comment

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

The formatter looks good to me, but I suggest we use docker compose for the task, so it's consistent with the other tasks and uses the same dedicated configuration.

I also changed the title of the PR, so it contains a brief description of changes for the changelog.

Taskfile.yml Outdated Show resolved Hide resolved
@codecov
Copy link

codecov bot commented Apr 21, 2023

Codecov Report

Merging #2485 (a679ba8) into main (91de3db) will increase coverage by 1.39%.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2485      +/-   ##
==========================================
+ Coverage   25.36%   26.75%   +1.39%     
==========================================
  Files         391      403      +12     
  Lines       18888    19976    +1088     
==========================================
+ Hits         4790     5345     +555     
- Misses      13561    14065     +504     
- Partials      537      566      +29     

see 18 files with indirect coverage changes

Flag Coverage Δ
integration 5.06% <ø> (?)
mongodb 5.06% <ø> (?)
unit 25.36% <ø> (ø)

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

Copy link
Contributor

@rumyantseva rumyantseva left a comment

Choose a reason for hiding this comment

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

Thanks a lot for contributing!

Taskfile.yml Show resolved Hide resolved
@rumyantseva rumyantseva enabled auto-merge (squash) April 24, 2023 16:40
@rumyantseva rumyantseva requested review from a team, w84thesun, chilagrow and noisersup April 24, 2023 16:40
Copy link
Contributor

@w84thesun w84thesun left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Member

@noisersup noisersup left a comment

Choose a reason for hiding this comment

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

Looks good to me!

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.

Thanks for your contribution!

@rumyantseva rumyantseva merged commit 11eb71d into FerretDB:main Apr 26, 2023
@AlekSi AlekSi added this to the v1.1.0 milestone May 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
code/chore Code maintenance improvements
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Use YAML formatter
7 participants