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

Fix puppetforge module feedback badges #9452

Closed

Conversation

smortex
Copy link

@smortex smortex commented Aug 8, 2023

The forge has stopped to collect user feedback a long time ago. However, it gained the capability to now run static analysis on modules and give it a score. This score has been work in progress for some time will probably change again in the future, and might include back user feedback in the future, but as this is moving quite slow, rather than waiting to unbreak this badge, we can switch to the current API endpoint that provide this scoring as a drop-in replacement for user feedback.

Since the badge is already labelled "score", this seems a reasonable move.

@github-actions
Copy link
Contributor

github-actions bot commented Aug 8, 2023

Warnings
⚠️ This PR modified service code for puppetforge but not its test code.
That's okay so long as it's refactoring existing code.
Messages
📖 ✨ Thanks for your contribution to Shields, @smortex!

Generated by 🚫 dangerJS against e9b8ad4

The forge has stopped to collect user feedback a long time ago.
However, it gained the capability to now run static analysis on modules
and give it a score.  This score has been work in progress for some time
will probably change again in the future, and might include back user
feedback in the future, but as this is moving quite slow, rather than
waiting to unbreak this badge, we can switch to the current API endpoint
that provide this scoring as a drop-in replacement for user feedback.

Since the badge is already labelled "score", this seems a reasonable
move.
@smortex smortex force-pushed the fix-puppetforge-module-feedback branch from e31f03c to e9b8ad4 Compare August 8, 2023 19:40
@chris48s
Copy link
Member

chris48s commented Aug 11, 2023

Thanks. When you say, they stopped collecting user feedback long ago, if I call
https://img.shields.io/puppetforge/f/camptocamp/openssl
for example, I still get a valid score back. So that is not "broken" to me. I guess that feedback is historical - no new data is being added to it - but it is still represents a true data point from the past.

I'd would say that "Puppet Forge Static Analysis Score" is just a completely new/different badge route. We can totally add that - feel free to modify the PR to just add that as a new badge.

It is a different thing from user feedback though. Particularly given that collecting feedback may be re-introduced at a later date, we're painting ourselves into a corner if we've re-used the route for a different concept.

@chris48s chris48s added the service-badge Accepted and actionable changes, features, and bugs label Aug 11, 2023
@PyvesB
Copy link
Member

PyvesB commented May 26, 2024

@smortex any chance you could address Chris's feedback in the coming weeks? :)

@smortex
Copy link
Author

smortex commented May 27, 2024

@PyvesB d'oh, I forgot about this. I opened #10201 to address it. Closing this one.

@smortex smortex closed this May 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
service-badge Accepted and actionable changes, features, and bugs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants