Skip to content

chore: add Scarf analytics pixel to README - #4840

Merged
gaius-qi merged 7 commits into
dragonflyoss:mainfrom
fcgxz2003:chore/scarf-analytics-pixel
Jul 9, 2026
Merged

chore: add Scarf analytics pixel to README#4840
gaius-qi merged 7 commits into
dragonflyoss:mainfrom
fcgxz2003:chore/scarf-analytics-pixel

Conversation

@fcgxz2003

@fcgxz2003 fcgxz2003 commented Jun 25, 2026

Copy link
Copy Markdown
Member

Description

Adds a Scarf tracking pixel to the bottom of the README so the Dragonfly community can measure anonymous, aggregated adoption trends and discovery sources for the project.

Related Issue

Motivation and Context

Screenshots (if appropriate)

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation Update (if none of the other choices apply)

Checklist

  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.

@fcgxz2003
fcgxz2003 requested review from a team as code owners June 25, 2026 06:42
@fcgxz2003
fcgxz2003 force-pushed the chore/scarf-analytics-pixel branch from 188f9df to 33f79c4 Compare June 25, 2026 06:43
@fcgxz2003 fcgxz2003 added the documentation Improvements or additions to documentation label Jun 25, 2026
Signed-off-by: fcgxz2003 <834756128@qq.com>
@fcgxz2003
fcgxz2003 force-pushed the chore/scarf-analytics-pixel branch from 33f79c4 to c45a034 Compare June 25, 2026 06:52
@fcgxz2003 fcgxz2003 closed this Jun 30, 2026
@fcgxz2003 fcgxz2003 reopened this Jun 30, 2026

@gaius-qi gaius-qi left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Use the img dom instead, otherwise an extra blank line will be displayed.

fcgxz2003 added 2 commits July 9, 2026 12:13
Signed-off-by: fcgxz2003 <834756128@qq.com>
Signed-off-by: fcgxz2003 <834756128@qq.com>
@codecov

codecov Bot commented Jul 9, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 28.12%. Comparing base (0b3cca4) to head (18cf2de).
⚠️ Report is 12 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #4840      +/-   ##
==========================================
+ Coverage   28.04%   28.12%   +0.08%     
==========================================
  Files         232      232              
  Lines       23148    23158      +10     
==========================================
+ Hits         6491     6513      +22     
+ Misses      16218    16197      -21     
- Partials      439      448       +9     
Flag Coverage Δ
unittests 28.12% <ø> (+0.08%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.
see 3 files with indirect coverage changes

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

gaius-qi added 4 commits July 9, 2026 13:01
- Add HTML comment to identify the Scarf analytics tracking pixel
- Hide the pixel image with `style="display:none"` to prevent
  layout impact
- Add empty `alt=""` attribute for accessibility compliance

Signed-off-by: Gaius <gaius.qi@gmail.com>
- Removes the hidden Scarf analytics image tag from README.md
- Eliminates passive tracking of users who view the repository
- No functional or documentation content is affected

Signed-off-by: Gaius <gaius.qi@gmail.com>
- Appends a 1×1 Scarf pixel image to the Code of Conduct line
- Enables anonymous download/view analytics via Scarf.sh
- Uses `no-referrer-when-downgrade` policy and hidden alt text
  to keep the tracker unobtrusive

Signed-off-by: Gaius <gaius.qi@gmail.com>
- Wraps the tracking pixel `<img>` tag with markdownlint disable/enable
  comments to suppress linting errors
- Adds blank lines around the section for proper markdown formatting
- Prevents CI/linting failures caused by the inline HTML image element

Signed-off-by: Gaius <gaius.qi@gmail.com>

@gaius-qi gaius-qi left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@gaius-qi
gaius-qi enabled auto-merge (squash) July 9, 2026 05:13
@gaius-qi
gaius-qi merged commit e965855 into dragonflyoss:main Jul 9, 2026
17 checks passed
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.

8 participants