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

add ghstats #1022

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

add ghstats #1022

wants to merge 4 commits into from

Conversation

vladkens
Copy link

Thanks for taking the time to suggest an addition to awesome-selfhosted!

To ensure your Pull Request is dealt with swiftly, please check the following (check the boxes [x]):

  • Submit one item per pull request. This eases reviewing and speeds up inclusion.
  • You have searched the repository for any relevant issues or PRs, including closed ones.
  • Any software you are adding is not already listed at any of awesome-sysadmin, staticgen.com, staticsitegenerators.net, dbdb.io.
  • The file you are adding is formatted as described in addition.md.
  • Demo links should only be used for interactive demos, i.e. not video demonstrations.
  • Comments and unused optional fields have been removed.
  • The file you are adding uses kebab-case file naming, for example my-awesome-software.yml.
  • Values for platform are the main server-side requirements for the software. Don't include frameworks or specific dialects.
  • Any software project you are adding to the list is actively maintained.
  • Any software project you are adding was first released more than 4 months ago.
  • Any software project you are adding has working installation instructions.
  • You understand that your Pull Request will be merged ~1 week after approval, to allow for further comments if needed.

@Ki-er Ki-er added addition Adding new software to the list reviewers wanted Awaiting reviews labels Dec 18, 2024
Copy link
Collaborator

@Ki-er Ki-er left a comment

Choose a reason for hiding this comment

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

Quick change.

software/ghstats.yml Outdated Show resolved Hide resolved
software/ghstats.yml Outdated Show resolved Hide resolved
Co-authored-by: Loymdayddaud <[email protected]>
Copy link
Collaborator

@kokomo123 kokomo123 left a comment

Choose a reason for hiding this comment

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

Seeing as the changes have been implemented, LGTM. No issues here 👍🏻

Comment on lines +9 to +11
tags:
- Personal Dashboards
- Analytics
Copy link
Member

@nodiscc nodiscc Dec 28, 2024

Choose a reason for hiding this comment

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

Suggested change
tags:
- Personal Dashboards
- Analytics
tags:
- Analytics

I know the scope of the https://awesome-selfhosted.net/tags/personal-dashboards.html tag is not clearly defined, but ghstats would look quite out-of-place there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
addition Adding new software to the list reviewers wanted Awaiting reviews
Development

Successfully merging this pull request may close these issues.

6 participants