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

new feature: icons number #1267

Merged
merged 7 commits into from
Mar 14, 2023
Merged

new feature: icons number #1267

merged 7 commits into from
Mar 14, 2023

Conversation

kilianpaquier
Copy link
Contributor

Double check these details before you open a PR

  • PR does not match another non-stale PR currently opened

Features

This PR closes #1218

Notes

Executed locally with python http.server. Feel free to write a review.

README.md Show resolved Hide resolved
docs/index.html Outdated Show resolved Hide resolved
docs/index.html Outdated Show resolved Hide resolved
docs/index.html Outdated Show resolved Hide resolved
@kilianpaquier kilianpaquier marked this pull request as draft July 14, 2022 21:06
@kilianpaquier kilianpaquier marked this pull request as ready for review July 15, 2022 08:26
@kilianpaquier kilianpaquier marked this pull request as draft July 22, 2022 09:36
@Snailedlt Snailedlt added the website Use this label for website related enhancements label Oct 15, 2022
README.md Outdated Show resolved Hide resolved
@kilianpaquier kilianpaquier marked this pull request as ready for review October 19, 2022 20:09
@Snailedlt Snailedlt added the hacktoberfest-accepted Accepted to be counted towards Hacktoberfest label Oct 31, 2022
Copy link
Contributor

@lunatic-fox lunatic-fox left a comment

Choose a reason for hiding this comment

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

I've suggested some changes and here's the preview. 🙂
preview

docs/assets/js/script.js Outdated Show resolved Hide resolved
docs/index.html Show resolved Hide resolved
docs/index.html Outdated Show resolved Hide resolved
@Panquesito7
Copy link
Member

Is this still being updated or?

@Snailedlt
Copy link
Collaborator

Is this still being updated or?

@Panquesito7 Yes, slowly but surely

@Snailedlt Snailedlt added the stale For PRs that haven't had any activity for a while and old issues label Mar 4, 2023
@Snailedlt
Copy link
Collaborator

Snailedlt commented Mar 4, 2023

@kilianpaquier let me know if you need any help with this :)
Labeling it as stale since it's been a while since the last update

@kilianpaquier
Copy link
Contributor Author

@kilianpaquier let me know if you need any help with this :) Labeling it as stale since it's been a while since the last update

Hello, does it still need some things to do ?

@Snailedlt
Copy link
Collaborator

@kilianpaquier let me know if you need any help with this :) Labeling it as stale since it's been a while since the last update

Hello, does it still need some things to do ?

@kilianpaquier Yes, see my last comment in the review comment from lunatic-fox

@kilianpaquier
Copy link
Contributor Author

@kilianpaquier let me know if you need any help with this :) Labeling it as stale since it's been a while since the last update

Hello, does it still need some things to do ?

@kilianpaquier Yes, see my last comment in the review comment from lunatic-fox

Ok I’m gonna take a look at it tomorrow !

@kilianpaquier
Copy link
Contributor Author

Hello @Snailedlt, it's done, avg is integer and float in title :). Should we squash the commits ?

@Snailedlt
Copy link
Collaborator

@kilianpaquier
Nice, we can squash merge it, so no need for you to squash :)
I'll review it when I get time

lunatic-fox
lunatic-fox previously approved these changes Mar 12, 2023
Copy link
Contributor

@lunatic-fox lunatic-fox left a comment

Choose a reason for hiding this comment

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

Looks good to me! 🚀

docs/index.html Outdated Show resolved Hide resolved
Co-authored-by: Josélio Júnior <76992016+lunatic-fox@users.noreply.github.com>
Copy link
Member

@Panquesito7 Panquesito7 left a comment

Choose a reason for hiding this comment

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

Amazing. Thanks! 🚀

Copy link
Collaborator

@Snailedlt Snailedlt left a comment

Choose a reason for hiding this comment

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

I haven't tested, but since both @Panquesito7 and @lunatic-fox has approved, I doubt there are any serious issues here. So I'll just approve it ✔️

Thank you again for the great contribution @kilianpaquier , they are a privilege to review!

@lunatic-fox lunatic-fox merged commit bda1033 into devicons:develop Mar 14, 2023
@Snailedlt Snailedlt mentioned this pull request Feb 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement hacktoberfest-accepted Accepted to be counted towards Hacktoberfest stale For PRs that haven't had any activity for a while and old issues website Use this label for website related enhancements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants