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

Github stuff for goose (Readme, CI, update go.work.sum) #47

Closed
wants to merge 2 commits into from

Conversation

deathowl
Copy link
Member

@deathowl deathowl commented Oct 5, 2023

What:

Why:

How:

Risks:

Checklist:

  • Added tests, if you've added code that should be tested
  • Updated the documentation, if you've changed APIs
  • Ensured the test suite passes
  • Made sure your code lints
  • Completed the Contributor License Agreement ("CLA")

@codecov-commenter
Copy link

codecov-commenter commented Oct 5, 2023

Codecov Report

All modified lines are covered by tests ✅

Comparison is base (05c7785) 58.83% compared to head (d8db971) 33.35%.

Additional details and impacted files
@@             Coverage Diff             @@
##             main      #47       +/-   ##
===========================================
- Coverage   58.83%   33.35%   -25.49%     
===========================================
  Files          72       17       -55     
  Lines        8318     1799     -6519     
===========================================
- Hits         4894      600     -4294     
+ Misses       3103     1159     -1944     
+ Partials      321       40      -281     
Files Coverage Δ
goose/query/query.go 43.06% <ø> (ø)
goose/query/validate.go 100.00% <ø> (ø)
goose/report/log.go 0.00% <ø> (ø)
goose/report/prometheus.go 100.00% <100.00%> (ø)
goose/report/util.go 100.00% <ø> (ø)

... and 64 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@facebook-github-bot
Copy link
Contributor

@deathowl has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@deathowl has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@deathowl has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@deathowl has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@deathowl merged this pull request in 5b38819.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants