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

Update automated themes preview workflow contributions guideline #2808

Merged
merged 2 commits into from
Jun 11, 2023

Conversation

qwerty541
Copy link
Collaborator

No description provided.

@qwerty541 qwerty541 requested a review from rickstaa June 11, 2023 02:56
@vercel
Copy link

vercel bot commented Jun 11, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
github-readme-stats ✅ Ready (Inspect) Visit Preview Jun 11, 2023 10:39am

@codecov
Copy link

codecov bot commented Jun 11, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (9105c74) 97.59% compared to head (7be1445) 97.59%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #2808   +/-   ##
=======================================
  Coverage   97.59%   97.59%           
=======================================
  Files          24       24           
  Lines        4822     4829    +7     
  Branches      453      453           
=======================================
+ Hits         4706     4713    +7     
  Misses        115      115           
  Partials        1        1           

see 1 file with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@rickstaa
Copy link
Collaborator

Good suggestion. I reworded the text and added a link to #1935. We can also add this in the README if we want 🤔.

Copy link
Collaborator Author

@qwerty541 qwerty541 left a comment

Choose a reason for hiding this comment

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

Good suggestion. I reworded the text and added a link to #1935. We can also add this in the README if we want thinking.

Nice

I think that adding this into PRs messages and contributing guide will be enough for now.

Can you approve this PR? I cannot approve my own 😞

@rickstaa rickstaa merged commit faf0c25 into master Jun 11, 2023
@rickstaa rickstaa deleted the ci_update_themes_preview_contribs_guideline branch June 11, 2023 13:22
devantler pushed a commit to devantler/github-readme-stats that referenced this pull request Sep 24, 2023
…raghazra#2808)

* Update automated themes preview workflow contributions guideline

* docs: link to top-issues dashboard

---------

Co-authored-by: rickstaa <[email protected]>
setdebarr pushed a commit to setdebarr/github-readme-stats that referenced this pull request Jan 12, 2024
…raghazra#2808)

* Update automated themes preview workflow contributions guideline

* docs: link to top-issues dashboard

---------

Co-authored-by: rickstaa <[email protected]>
jacobbexten pushed a commit to jacobbexten/github-readme-stats that referenced this pull request Nov 6, 2024
…raghazra#2808)

* Update automated themes preview workflow contributions guideline

* docs: link to top-issues dashboard

---------

Co-authored-by: rickstaa <[email protected]>
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