Skip to content

Conversation

@nvgscientist
Copy link
Contributor

  • add SECURITY.md in rare event of security issue
  • add .github/pull_request_template.md PR template
  • update readme, formatting, verbiage
  • prevent workflow from on changes to markdown files

@nvgscientist nvgscientist added the documentation Improvements or additions to documentation label Jan 20, 2025
@nvgscientist nvgscientist self-assigned this Jan 20, 2025
@github-actions
Copy link

github-actions bot commented Jan 20, 2025

badge

Code Coverage Summary

Filename                    Stmts    Miss  Cover    Missing
------------------------  -------  ------  -------  ------------------------------------------------------------------------------
rustmaps.go                     1       0  100.00%
pkg/api/status.go              41       6  85.37%   90, 99-100, 109-110, 119
pkg/api/generate.go            98      37  62.24%   65, 72, 92-93, 115, 125-138, 156, 163, 173-174, 183-184, 191-205, 210, 221-226
pkg/api/limits.go              27      10  62.96%   50-51, 60-61, 70-71, 78-79, 84-85
pkg/api/main.go                11       0  100.00%
pkg/rustmaps/main.go           43       4  90.70%   52, 70, 111-112
pkg/rustmaps/setters.go         3       0  100.00%
pkg/rustmaps/auth.go           15       4  73.33%   26-27, 34-35
pkg/rustmaps/config.go         13       6  53.85%   18-27, 34
pkg/rustmaps/helpers.go        21       0  100.00%
pkg/rustmaps/import.go         98      13  86.73%   34, 45, 50, 106-107, 159-160, 166-167, 179-180, 187-188
pkg/rustmaps/download.go       66      14  78.79%   30-31, 48-49, 55-56, 89-90, 102-103, 106-107, 110-111
pkg/rustmaps/generate.go       29       5  82.76%   25, 29, 35-36, 51
pkg/rustmaps/getters.go         5       0  100.00%
pkg/types/main.go              38       2  94.74%   106, 114
TOTAL                         509     101  80.16%

Results for commit: acc2099

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

Copy link
Contributor

@heavyscientist heavyscientist left a comment

Choose a reason for hiding this comment

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

lgtm

@nvgscientist nvgscientist merged commit 5c4c8b7 into main Jan 20, 2025
3 checks passed
@nvgscientist nvgscientist deleted the staging branch January 20, 2025 23:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants