argo-workflows icon indicating copy to clipboard operation
argo-workflows copied to clipboard

fix: replace deprecated `bouk/staticfiles` with native Go `embed`. fixes #11654

Open sonbui00 opened this issue 1 year ago • 9 comments

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

sonbui00 avatar Aug 29 '23 12:08 sonbui00

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

stale[bot] avatar Sep 17 '23 02:09 stale[bot]

@agilgur5 Should I do anything to go forward for this?

sonbui00 avatar Sep 20 '23 00:09 sonbui00

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.

agilgur5 avatar Sep 20 '23 04:09 agilgur5

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!)

agilgur5 avatar Sep 20 '23 04:09 agilgur5

@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 avatar Jun 28 '24 03:06 juliev0

@juliev0 I'll check it

sonbui00 avatar Jun 29 '24 14:06 sonbui00

let me know when you've fixed the codegen and lint issues and we can merge this

juliev0 avatar Jul 03 '24 17:07 juliev0

@juliev0 codegen and lint issues have been fixed. help me review

sonbui00 avatar Jul 04 '24 08:07 sonbui00

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?

juliev0 avatar Jul 04 '24 14:07 juliev0