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 usage trend #755

Closed
wants to merge 4 commits into from
Closed

add usage trend #755

wants to merge 4 commits into from

Conversation

cwtuan
Copy link

@cwtuan cwtuan commented May 26, 2024

I'm an active user of sequelize and I'm really impressed with its capabilities. As sequelize's popularity has grown significantly in recent years, I believe it would be helpful to include a resource in the README to assist new users who are choosing an nodejs ORM solution.

I believe this can provide value to new users and the sequelize community. Thank you for your consideration

@cwtuan cwtuan requested a review from a team as a code owner May 26, 2024 05:17
@cwtuan cwtuan requested review from ephys and WikiRik May 26, 2024 05:17
@WikiRik
Copy link
Member

WikiRik commented May 26, 2024

Thanks for this PR! Starting with v7, we release new versions under @sequelize/core. Is there a way of combining those usages in one graph to display on the page?

@cwtuan
Copy link
Author

cwtuan commented May 27, 2024

@WikiRik do you mean something like this?

image

@ephys
Copy link
Member

ephys commented Nov 16, 2024

@cwtuan I think @WikiRik means adding @sequelize/core to the list (the others are not as important, core is the only one that is mandatory)

Though I don't think they can be merged, so the graph would look like this:

image

@cwtuan
Copy link
Author

cwtuan commented Nov 16, 2024

Thanks for the suggestion!

Based on your feedback, I separated the download trends for sequelize@6 and sequelize@7 into two individual charts instead of combining them into one. The main reason is that, while the usage of sequelize@7 has indeed grown rapidly over the past one year, it is still relatively lower compared to sequelize@6. Combining the two trends in a single chart might create a misleading impression that sequelize@7 has almost no usage.

By presenting these two charts separately, users can clearly observe the significant growth in sequelize@7 usage over the past year. This trend also reflects the ongoing migration of existing users to sequelize@7.

Additionally, I’m the main maintainer of npm-compare.com. If you have any other feature requests or needs, I’d be happy to implement them.

Finally, I also want to express my gratitude to you guys for the effort in developing sequelize@7.

Here is preview:

image

@ephys
Copy link
Member

ephys commented Nov 16, 2024

This looks really nice! Thank you 🥰

I’m the main maintainer of npm-compare.com

Oh! I didn't know. npm-compare looks great. It was nice to see that Sequelize is still increasing in use

I think this would be better suited for the homepage than the guide & tutorial index. I'll see if I can add it there

Edit: This is what it could look like - #772

@cwtuan
Copy link
Author

cwtuan commented Nov 17, 2024

I think this is an awesome idea! Putting the charts on the homepage is a great way to let more people know that Sequelize is still growing fast. As a Sequelize user, I’m really happy I could help contribute to this.

Since there’s a new implementation now, I’ve closed my PR. Thanks again for all the effort you’ve put into this!

@cwtuan cwtuan closed this Nov 17, 2024
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.

3 participants