Skip to content

[WIP] Makefile changes#120

Draft
rafie wants to merge 1 commit into
masterfrom
rafi-build1
Draft

[WIP] Makefile changes#120
rafie wants to merge 1 commit into
masterfrom
rafi-build1

Conversation

@rafie
Copy link
Copy Markdown
Contributor

@rafie rafie commented Jul 18, 2021

No description provided.

@sonarqubecloud
Copy link
Copy Markdown

SonarCloud Quality Gate failed.

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell C 1 Code Smell

No Coverage information No Coverage information
0.0% 0.0% Duplication

}

func init() {
func _init() {
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

U1000: func _init is unused
(at-me in a reply with help or ignore)

}

func init() {
func _init() {
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

deadcode: _init is unused
(at-me in a reply with help or ignore)

@codecov
Copy link
Copy Markdown

codecov Bot commented Jul 18, 2021

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.72%. Comparing base (dbcb367) to head (ef1ab30).
⚠️ Report is 35 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #120      +/-   ##
==========================================
- Coverage   76.90%   76.72%   -0.19%     
==========================================
  Files          13       13              
  Lines        1104     1104              
==========================================
- Hits          849      847       -2     
- Misses        198      199       +1     
- Partials       57       58       +1     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@rafie rafie changed the title Makefile changes [WIP] Makefile changes Jul 18, 2021
@rafie rafie marked this pull request as draft July 18, 2021 16:11
Comment thread Makefile

TEST ?= Test
ifeq ($(VERBOSE),1)
TEST_FLAGS += -v
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

we should add -count=1 to TEST_FLAGS to ensure multiple calls to test do indeed trigger the tests.

Comment thread Makefile
@@ -60,3 +78,11 @@ godoc:
echo "Open browser tab on localhost:6060"
$(GODOC)

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

if possible can we take this chance to also add the microbenchamrks rule @rafie ?
sample one:
https://github.com/HdrHistogram/hdrhistogram-go/blob/master/Makefile#L40

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants