Skip to content

Remove insecure workflow#535

Open
MatthewHambley wants to merge 4 commits intomainfrom
RemovePicasso
Open

Remove insecure workflow#535
MatthewHambley wants to merge 4 commits intomainfrom
RemovePicasso

Conversation

@MatthewHambley
Copy link
Collaborator

This workflow was flagged as insecure by an automatic check. I'm not sure what it's for, some sort of container I think. So the simplest solution is to get rid of it.

This workflow was flagged as insecure by an automatic check. I'm not sure what it's for, some sort of container I think. So the simplest solution is to get rid of it.
@MatthewHambley MatthewHambley requested a review from t00sa January 12, 2026 16:36
@MatthewHambley MatthewHambley added the CI GitHub Actions related changes label Jan 12, 2026
@t00sa
Copy link
Contributor

t00sa commented Jan 12, 2026

Originally added three years ago by PR #206. We don't use the singularity images, so I don't think there's a reason to keep this.

t00sa
t00sa previously approved these changes Jan 12, 2026
Copy link
Contributor

@t00sa t00sa left a comment

Choose a reason for hiding this comment

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

Approving but waiting for feedback from @yaswant before merging.

@yaswant
Copy link
Collaborator

yaswant commented Jan 15, 2026

Before removing this GA workflow, I'd like to see if picasso can actually build and push the container to our artifactory. I'll open a separate PR for testing.

yaswant
yaswant previously approved these changes Feb 19, 2026
Copy link
Collaborator

@yaswant yaswant 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 tried several approaches but haven't been able to get this action to work. I think it's fine to remove this workflow from the repository.

@MatthewHambley MatthewHambley dismissed stale reviews from yaswant and t00sa via 1ab934b February 27, 2026 10:46
@MatthewHambley MatthewHambley requested a review from t00sa February 27, 2026 10:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI GitHub Actions related changes

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

3 participants