falcosecurity / test-infra

Falco workflow & testing infrastructure
https://prow.falco.org
Apache License 2.0
31 stars 109 forks source link

poiana comment has a broken link to contributing guide #430

Closed Rajakavitha1 closed 2 years ago

Rajakavitha1 commented 3 years ago

What is wrong with the website or needs to be cleaned?

The link now points to https://github.com/falcosecurity/falco-website/blob/master/CONTRIBUTING.md and results in 404.

Update the link to https://github.com/falcosecurity/.github/blob/master/CONTRIBUTING.md

Screenshot 2020-11-22 at 3 30 18 PM
poiana commented 3 years ago

Issues go stale after 90d of inactivity.

Mark the issue as fresh with /remove-lifecycle stale.

Stale issues rot after an additional 30d of inactivity and eventually close.

If this issue is safe to close now please do so with /close.

Provide feedback via https://github.com/falcosecurity/community.

/lifecycle stale

poiana commented 3 years ago

Stale issues rot after 30d of inactivity.

Mark the issue as fresh with /remove-lifecycle rotten.

Rotten issues close after an additional 30d of inactivity.

If this issue is safe to close now please do so with /close.

Provide feedback via https://github.com/falcosecurity/community.

/lifecycle rotten

leogr commented 3 years ago

/remove-lifecycle rotten

leogr commented 3 years ago

We have to take care of our beloved @poiana :)

PS Since it's a prow's plugin related issue I'm moving this issue to test-infra

leogr commented 3 years ago

/remove-lifecycle stale

leogr commented 3 years ago

Hey

it seems to me that #431 fixed another issue, since the "contributing guide" link is still pointing to a wrong URL. My test here: https://github.com/falcosecurity/falco-website/pull/474#issuecomment-871200881

It was reported again in https://github.com/falcosecurity/falco-website/issues/473

/reopen

leogr commented 3 years ago

Additional context:

atarax665 commented 3 years ago

Additional context:

Yeah, I too figured out the same. Only feasible way to resolve this issue from my perspective is,

I assume that the organization has a common Contributing Guide (in /.github) for all the repositories, so we can change targetURL := fmt.Sprintf("https://github.com/%s/%s/blob/master/CONTRIBUTING.md", org, repo) to targetURL := fmt.Sprintf("https://github.com/%s/.github/blob/master/CONTRIBUTING.md", org) or maybe we should keep the targetURL static as targetURL = "https://github.com/falcosecurity/.github/blob/master/CONTRIBUTING.md" because org will be falcosecurity for all the repositories.

PS: Though falcosidekick has its own CONTRIBUTING.md somehow.

Another way out can be to add a CONTRIBUTING.md in each repository as that would generate a valid URL through existing DCO plugin (check here but I guess that will defeat the whole purpose of having falcosecurity/.github repository.

@leogr Please do share your insights.

leogr commented 3 years ago

I assume that the organization has a common Contributing Guide (in /.github) for all the repositories, so we can change targetURL := fmt.Sprintf("https://github.com/%s/%s/blob/master/CONTRIBUTING.md", org, repo) to targetURL := fmt.Sprintf("https://github.com/%s/.github/blob/master/CONTRIBUTING.md", org) or maybe we should keep the targetURL static as targetURL = "https://github.com/falcosecurity/.github/blob/master/CONTRIBUTING.md" because org will be falcosecurity for all the repositories.

I believe the main issue here is that we have to host and maintain our own customized DCO plugin. Once we have that, we could use a static URL or make it configurable on a per-repo basis.

PS: Though falcosidekick has its own CONTRIBUTING.md somehow.

Another way out can be to add a CONTRIBUTING.md in each repository as that would generate a valid URL through existing DCO plugin (check here but I guess that will defeat the whole purpose of having falcosecurity/.github repository.

The original idea was to have just one centralized CONTRIBUTING.md. And btw I still like that idea since it requires less maintenance. :smile_cat: I guess that once we solve the DCO plugin issue, even falcosidekick could use the centralized CONTRIBUTING.md. @falcosecurity/falcosidekick-maintainers wdyt?

Issif commented 3 years ago

I guess that once we solve the DCO plugin issue, even falcosidekick could use the centralized CONTRIBUTING.md. @falcosecurity/falcosidekick-maintainers wdyt?

Ok for me

poiana commented 2 years ago

Issues go stale after 90d of inactivity.

Mark the issue as fresh with /remove-lifecycle stale.

Stale issues rot after an additional 30d of inactivity and eventually close.

If this issue is safe to close now please do so with /close.

Provide feedback via https://github.com/falcosecurity/community.

/lifecycle stale

leogr commented 2 years ago

/remove-lifecycle stale

zuc commented 2 years ago

I just opened https://github.com/kubernetes/test-infra/pull/25495 to address this, let's see if it goes through 🙂

zuc commented 2 years ago

/assign @zuc

poiana commented 2 years ago

Issues go stale after 90d of inactivity.

Mark the issue as fresh with /remove-lifecycle stale.

Stale issues rot after an additional 30d of inactivity and eventually close.

If this issue is safe to close now please do so with /close.

Provide feedback via https://github.com/falcosecurity/community.

/lifecycle stale

maxgio92 commented 2 years ago

/remove-lifecycle stale

poiana commented 2 years ago

Issues go stale after 90d of inactivity.

Mark the issue as fresh with /remove-lifecycle stale.

Stale issues rot after an additional 30d of inactivity and eventually close.

If this issue is safe to close now please do so with /close.

Provide feedback via https://github.com/falcosecurity/community.

/lifecycle stale

zuc commented 2 years ago

The upstream PR just got merged yesterday (https://github.com/kubernetes/test-infra/pull/25495) so we can finally go ahead and fix this.

/remove-lifecycle stale