argo-workflows
argo-workflows copied to clipboard
fix: replace deprecated `bouk/staticfiles` with native Go `embed`. fixes #11654
Fixes #11654
Motivation
- bouk/staticfiles is deprecated
- Sometime server/static/files.go is added to git commit unexpectedly
Modifications
- remove bouk/staticfiles, use standard lib
Verification
- make test
- manual check
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.
@agilgur5 Should I do anything to go forward for this?
Usually the process is that someone does initial reviews and if it's not an approver, we wait for an approver to then approve and merge.
Workflows doesn't have many active approvers so there's occasionally a backlog as there is now. I also have some PRs that have been open for 3+ weeks 😕 That is probably the longest wait I've seen in the past few months though, fwiw (I've only been actively contributing the past few, so can't say beyond that).
I had initially deferred review to someone who may know embed
etc better (I have used it exactly once as part of a CLI, not a server), but Workflows might not actually have any contributors familiar with it, so maybe I should do first review.
For context, I am currently a Member (though may become a Reviewer at the next membership meeting). Lead > Approver > Reviewer > Member. You can see the Argo governance docs here. You are also welcome to request Membership if you're interested 😉 (we can probably find a sponsor as well; sponsors must be approvers+, though support from others in the community like me helps!)
@sonbui00 sorry this ended up being delayed. I see @isubasinghe approved this awhile back. @isubasinghe is now an Approver and can merge. Do you want to fix the merge conflicts and get this merged still?
@juliev0 I'll check it
let me know when you've fixed the codegen and lint issues and we can merge this
@juliev0 codegen and lint issues have been fixed. help me review
Sorry @isubasinghe - it looks like your original review has been removed with the fixes @sonbui00 made . Can you re-review/merge when you have time?