Skip to content

feat: Update the list of sample#3545

Merged
SC-Samir merged 1 commit intoScalingo:masterfrom
SC-Samir:master
Feb 23, 2026
Merged

feat: Update the list of sample#3545
SC-Samir merged 1 commit intoScalingo:masterfrom
SC-Samir:master

Conversation

@SC-Samir
Copy link
Contributor

@SC-Samir SC-Samir commented Feb 2, 2026

This commit permit to update the list of sample that we display on our website.

This commit permit to update the list of sample that we display
on our website.
@SC-Samir SC-Samir self-assigned this Feb 2, 2026
Copy link
Member

@EtienneM EtienneM left a comment

Choose a reason for hiding this comment

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

Same comment as with the Terraform PR: let's wait for IST to decide which samples they wanna keep.

@SC-Samir SC-Samir added the on hold Something is currently preventing work (decision waiting, third party, etc) label Feb 9, 2026
Copy link
Member

@EtienneM EtienneM 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 seen the final decision about which samples we keep and which one we drop. Hence I can't approve this

@matthieu526-scalingo
Copy link
Contributor

I haven't seen the final decision about which samples we keep and which one we drop. Hence I can't approve this

On the file where the discussion happened, all samples in doubt were discussed and comments resolved. In the end it was decided to drop all samples that where questioned, as no one uses them. In any case as they are only archived, they can still be used for testing purposes, just not maintained. Does this answer your question ?

Copy link
Member

@EtienneM EtienneM left a comment

Choose a reason for hiding this comment

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

@SC-Samir did you made yourself the owner of the samples on GitHub so that the Dependabot PRs are assigned to you?

@SC-Samir
Copy link
Contributor Author

I need to do it! so please don't merge this PR

@SC-Samir SC-Samir merged commit 5efc751 into Scalingo:master Feb 23, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

on hold Something is currently preventing work (decision waiting, third party, etc) samples

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants