Skip to content

Add community badges to README - #16

Merged
kevin-dimichel merged 2 commits into
voxel51:mainfrom
Burhan-Q:update-community-badges
Mar 24, 2026
Merged

kevin-dimichel merged 2 commits into
voxel51:mainfrom
Burhan-Q:update-community-badges

Conversation

@Burhan-Q

Copy link
Copy Markdown
Contributor

Summary

  • Adds a consistent set of community/social badges (Discord, Hugging Face,
    Blog, Newsletter, LinkedIn, Twitter/X, Medium) to the README
  • Removes any outdated Slack community references
  • Badges are placed on their own row, separate from code/developer badges

This is part of an org-wide effort to ensure consistent community links
across all Voxel51 repositories.

@Burhan-Q
Burhan-Q requested a review from a team as a code owner March 24, 2026 12:51

@kevin-dimichel kevin-dimichel left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The pre-commit PR check failed because the PR source branch is a fork. As an employee, you don't need to fork repos within our org, even public ones.

I cloned your fork and ran the pre-commit checks:

[fiftyone-plugin-examples]$ pre-commit run -a
check for added large files..............................................Passed
check for case conflicts.................................................Passed
check that scripts with shebangs are executable..........................Passed
check yaml...............................................................Passed
fix end of files.........................................................Passed
mixed line ending........................................................Passed
pretty format json...................................(no files to check)Skipped
trim trailing whitespace.................................................Passed
No-tabs checker..........................................................Passed
codespell................................................................Passed
markdownlint.............................................................Failed
- hook id: markdownlint
- exit code: 1

README.md:3:1 MD033/no-inline-html Inline HTML [Element: div]

markdownlint-fix.........................................................Failed
- hook id: markdownlint-fix
- exit code: 1

README.md:3:1 MD033/no-inline-html Inline HTML [Element: div]

Detect secrets...........................................................Passed
yamllint.................................................................Passed
Lint GitHub Actions workflow files.......................................Passed
black................................................(no files to check)Skipped
isort (python).......................................(no files to check)Skipped
shfmt................................................(no files to check)Skipped
shellcheck...........................................(no files to check)Skipped

After adding the suggestions, I ran the pre-commit again:

[fiftyone-plugin-examples]$ git add README.md
[fiftyone-plugin-examples]$ pre-commit run -a
check for added large files..............................................Passed
check for case conflicts.................................................Passed
check that scripts with shebangs are executable..........................Passed
check yaml...............................................................Passed
fix end of files.........................................................Passed
mixed line ending........................................................Passed
pretty format json...................................(no files to check)Skipped
trim trailing whitespace.................................................Passed
No-tabs checker..........................................................Passed
codespell................................................................Passed
markdownlint.............................................................Passed
markdownlint-fix.........................................................Passed
Detect secrets...........................................................Passed
yamllint.................................................................Passed
Lint GitHub Actions workflow files.......................................Passed
black................................................(no files to check)Skipped
isort (python).......................................(no files to check)Skipped
shfmt................................................(no files to check)Skipped
shellcheck...........................................(no files to check)Skipped

Comment thread README.md Resolved
Comment thread README.md Resolved
@Burhan-Q

Burhan-Q commented Mar 24, 2026 •

Copy link
Copy Markdown
Contributor Author

@kevin-dimichel I opened 12 similar PRs today all from forks. That's because on those repos I did not have permission to create a branch. Happy to rework all of those PRs from a branch once given permission. See thread in #community channel on Slack, as I'm not the only one missing permissions

Co-authored-by: Kevin DiMichel <56850465+kevin-dimichel@users.noreply.github.com>

@kevin-dimichel kevin-dimichel left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

After pulling the latest changes, the pre-commit now passes:

[fiftyone-plugin-examples]$ git pull
remote: Enumerating objects: 5, done.
remote: Counting objects: 100% (5/5), done.
remote: Compressing objects: 100% (3/3), done.
remote: Total 3 (delta 2), reused 0 (delta 0), pack-reused 0 (from 0)
Unpacking objects: 100% (3/3), 1018 bytes | 203.00 KiB/s, done.
From github.com:Burhan-Q/fiftyone-plugin-examples
   8b104d7..752e185  update-community-badges -> origin/update-community-badges
Updating 8b104d7..752e185
Created autostash: 67041eb
Fast-forward
 README.md | 2 ++
 1 file changed, 2 insertions(+)
Applied autostash.
[fiftyone-plugin-examples]$ pre-commit run -a
check for added large files..............................................Passed
check for case conflicts.................................................Passed
check that scripts with shebangs are executable..........................Passed
check yaml...............................................................Passed
fix end of files.........................................................Passed
mixed line ending........................................................Passed
pretty format json...................................(no files to check)Skipped
trim trailing whitespace.................................................Passed
No-tabs checker..........................................................Passed
codespell................................................................Passed
markdownlint.............................................................Passed
markdownlint-fix.........................................................Passed
Detect secrets...........................................................Passed
yamllint.................................................................Passed
Lint GitHub Actions workflow files.......................................Passed
black................................................(no files to check)Skipped
isort (python).......................................(no files to check)Skipped
shfmt................................................(no files to check)Skipped
shellcheck...........................................(no files to check)Skipped

@kevin-dimichel
kevin-dimichel merged commit a398ee1 into voxel51:main Mar 24, 2026
1 of 2 checks passed
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