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

Update CONTRIBUTING.md with typo fix and a tiny correction #574

Merged
merged 1 commit into from
May 9, 2022

Conversation

rumyantseva
Copy link
Member

@rumyantseva rumyantseva commented May 9, 2022

The original purpose of this change was to correct a typo.

Though, after re-reading the text, I believe it's better to call cmd directory, not package. All the packages inside cmd are called main ;)

It could be a bit opinionated, so feel free to reject if you disagree.

@rumyantseva rumyantseva requested a review from AlekSi as a code owner May 9, 2022 18:42
@ferretdb-bot
Copy link
Member

ferretdb-bot commented May 9, 2022

CLA assistant check
All committers have signed the CLA.

@rumyantseva rumyantseva added the documentation Something user-visible is badly or not documented label May 9, 2022
@AlekSi AlekSi added this to the v0.2.1 milestone May 9, 2022
@AlekSi AlekSi enabled auto-merge (squash) May 9, 2022 19:17
@AlekSi
Copy link
Member

AlekSi commented May 9, 2022

I only disagree that this PR is opinionated: the text was factually incorrect :)

@codecov
Copy link

codecov bot commented May 9, 2022

Codecov Report

Merging #574 (f1fd721) into main (331039b) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #574   +/-   ##
=======================================
  Coverage   61.03%   61.03%           
=======================================
  Files         121      121           
  Lines        6650     6650           
=======================================
  Hits         4059     4059           
  Misses       2081     2081           
  Partials      510      510           
Flag Coverage Δ
FerretDB 54.51% <ø> (ø)
MongoDB 6.70% <ø> (ø)
integration 54.58% <ø> (ø)
unit 26.99% <ø> (ø)

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

@AlekSi AlekSi merged commit 6174ea3 into main May 9, 2022
@AlekSi AlekSi deleted the docs-code-overview branch May 9, 2022 19:20
@AlekSi AlekSi assigned rumyantseva and unassigned AlekSi May 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Something user-visible is badly or not documented
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants