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 documentation about fjson package #262

Merged
merged 2 commits into from
Jan 15, 2022
Merged

Update documentation about fjson package #262

merged 2 commits into from
Jan 15, 2022

Conversation

AlekSi
Copy link
Member

@AlekSi AlekSi commented Jan 14, 2022

Closes #182.

@AlekSi AlekSi added code/chore Code maintenance improvements documentation Something user-visible is badly or not documented labels Jan 14, 2022
@codecov
Copy link

codecov bot commented Jan 14, 2022

Codecov Report

Merging #262 (4fd04dc) into main (e4e9e24) will not change coverage.
The diff coverage is 60.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #262   +/-   ##
=======================================
  Coverage   51.04%   51.04%           
=======================================
  Files         109      109           
  Lines        4958     4958           
=======================================
  Hits         2531     2531           
  Misses       2029     2029           
  Partials      398      398           
Impacted Files Coverage Δ
internal/bson/bson.go 0.00% <ø> (ø)
internal/bson/document.go 43.91% <ø> (ø)
internal/clientconn/conn.go 0.00% <ø> (ø)
internal/fjson/fjson.go 85.52% <ø> (ø)
internal/handlers/msg_getcmdlineopts.go 0.00% <0.00%> (ø)
internal/handlers/query_cmd.go 0.00% <0.00%> (ø)
internal/types/array.go 66.10% <ø> (ø)
internal/types/types.go 94.11% <ø> (ø)
internal/wire/op_msg.go 46.00% <ø> (ø)
internal/wire/op_query.go 21.42% <ø> (ø)
... and 3 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e4e9e24...4fd04dc. Read the comment docs.

CONTRIBUTING.md Outdated Show resolved Hide resolved
OpenSauce
OpenSauce previously approved these changes Jan 14, 2022
Co-authored-by: Leigh <lawleycobhc@gmail.com>
@AlekSi AlekSi merged commit 8e6d03b into main Jan 15, 2022
@AlekSi AlekSi deleted the fjson-cleanup branch January 15, 2022 06:22
@AlekSi AlekSi added this to the v0.0.6 milestone Jan 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
code/chore Code maintenance improvements documentation Something user-visible is badly or not documented
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Extract fjson package
3 participants