eclipse-platform / .eclipsefdn

Repository to host configurations related to the Eclipse Foundation.
https://eclipse-platform.github.io/.eclipsefdn/
0 stars 2 forks source link

Archive deprecated repos #3

Closed merks closed 10 months ago

merks commented 10 months ago

Mark repositories that have been merged into other repositories as archived.

github-actions[bot] commented 10 months ago
Diff for 2be8e43d3a2b806fa95a9037544f4fa5289fcd2f: ```diff Printing local diff for configuration at '/home/runner/work/.eclipsefdn/.eclipsefdn/otterdog-configs/otterdog.json' Actions are indicated with the following symbols: + create ! modify ! forced update - delete Organization eclipse-platform[id=eclipse-platform] there have been 46 validation infos, enable verbose output with '-v' to to display them. ! repository[name="eclipse.platform.common"] { ! archived = false -> true ! } ! repository[name="eclipse.platform.debug"] { ! archived = false -> true ! } ! repository[name="eclipse.platform.releng"] { ! archived = false -> true ! } ! repository[name="eclipse.platform.resources"] { ! archived = false -> true ! } ! repository[name="eclipse.platform.runtime"] { ! archived = false -> true ! } ! repository[name="eclipse.platform.team"] { ! archived = false -> true ! } ! repository[name="eclipse.platform.text"] { ! archived = false -> true ! } ! repository[name="eclipse.platform.ua"] { ! archived = false -> true ! } ! repository[name="www.eclipse.org-eclipse-news"] { ! archived = false -> true ! } Plan: 0 to add, 9 to change, 0 to delete. ``` ```diff Showing diff to a canonical version of the configuration at '/home/runner/work/.eclipsefdn/.eclipsefdn/otterdog-configs/otterdog.json' Organization eclipse-platform[id=eclipse-platform] --- original +++ canonical @@ -41,6 +41,7 @@ web_commit_signoff_required: false } orgs.newRepo('eclipse.platform.common') { + allow_merge_commit: false archived: true branch_protection_rules: [ orgs.newBranchProtectionRule('R*maintenance') { @@ -61,13 +62,16 @@ } ] default_branch: "master" + dependabot_alerts_enabled: false has_projects: false has_wiki: false homepage: "https://www.eclipse.org/eclipse" secret_scanning: "disabled" + secret_scanning_push_protection: "disabled" web_commit_signoff_required: false } orgs.newRepo('eclipse.platform.debug') { + allow_merge_commit: false archived: true branch_protection_rules: [ orgs.newBranchProtectionRule('R*maintenance') { @@ -87,6 +91,7 @@ has_wiki: false homepage: "https://www.eclipse.org/eclipse/" secret_scanning: "disabled" + secret_scanning_push_protection: "disabled" topics+: [ "debug" "eclipse" @@ -118,6 +123,7 @@ web_commit_signoff_required: false } orgs.newRepo('eclipse.platform.releng') { + allow_merge_commit: false archived: true branch_protection_rules: [ orgs.newBranchProtectionRule('R*maintenance') { @@ -132,9 +138,11 @@ } ] default_branch: "master" + dependabot_alerts_enabled: false has_discussions: true has_projects: false secret_scanning: "disabled" + secret_scanning_push_protection: "disabled" web_commit_signoff_required: false } orgs.newRepo('eclipse.platform.releng.aggregator') { @@ -209,6 +217,7 @@ web_commit_signoff_required: false } orgs.newRepo('eclipse.platform.resources') { + allow_merge_commit: false archived: true branch_protection_rules: [ orgs.newBranchProtectionRule('R*maintenance') { @@ -229,9 +238,11 @@ has_projects: false has_wiki: false secret_scanning: "disabled" + secret_scanning_push_protection: "disabled" web_commit_signoff_required: false } orgs.newRepo('eclipse.platform.runtime') { + allow_merge_commit: false archived: true branch_protection_rules: [ orgs.newBranchProtectionRule('R*maintenance') { @@ -249,6 +260,7 @@ has_projects: false has_wiki: false secret_scanning: "disabled" + secret_scanning_push_protection: "disabled" web_commit_signoff_required: false } orgs.newRepo('eclipse.platform.swt') { @@ -330,6 +342,7 @@ web_commit_signoff_required: false } orgs.newRepo('eclipse.platform.team') { + allow_merge_commit: false archived: true branch_protection_rules: [ orgs.newBranchProtectionRule('R*maintenance') { @@ -344,12 +357,15 @@ } ] default_branch: "master" + dependabot_alerts_enabled: false has_projects: false has_wiki: false secret_scanning: "disabled" + secret_scanning_push_protection: "disabled" web_commit_signoff_required: false } orgs.newRepo('eclipse.platform.text') { + allow_merge_commit: false archived: true branch_protection_rules: [ orgs.newBranchProtectionRule('R*maintenance') { @@ -367,9 +383,11 @@ has_projects: false has_wiki: false secret_scanning: "disabled" + secret_scanning_push_protection: "disabled" web_commit_signoff_required: false } orgs.newRepo('eclipse.platform.ua') { + allow_merge_commit: false archived: true branch_protection_rules: [ orgs.newBranchProtectionRule('R*maintenance') { @@ -384,9 +402,11 @@ } ] default_branch: "master" + dependabot_alerts_enabled: false has_projects: false has_wiki: false secret_scanning: "disabled" + secret_scanning_push_protection: "disabled" web_commit_signoff_required: false } orgs.newRepo('eclipse.platform.ui') { @@ -489,6 +509,7 @@ ] } orgs.newRepo('www.eclipse.org-eclipse-news') { + allow_merge_commit: false archived: true branch_protection_rules: [ orgs.newBranchProtectionRule('master') { @@ -501,6 +522,7 @@ has_projects: false has_wiki: false secret_scanning: "disabled" + secret_scanning_push_protection: "disabled" web_commit_signoff_required: false webhooks: [ orgs.newRepoWebhook('https://ci.eclipse.org/releng/github-webhook/') { ```
merks commented 10 months ago

@akurtakov

If you look at Mylyn

https://github.com/orgs/eclipse-mylyn/repositories

you can see that some (most) repositories are marked as archived:

image

I think that because they are marked that way like via this archived: true property:

https://github.com/eclipse-mylyn/.eclipsefdn/blob/15e9de788033b1f90016ebb503e8b122b4857072/otterdog/eclipse-mylyn.jsonnet#L72-L77

Probably it's good for the platform to do this as well so that one can see at a glance what's not relevant anymore.

@netomi Please wait for @akurtakov to agree to this change.

netomi commented 10 months ago

indeed, setting the property archive to true has the effect as you point out.

akurtakov commented 10 months ago

Repositories have not been archieved as that would make it impossible to backport fixes for old release branches and some companies still do that. Please correct me if I'm wrong and one can still push to archived repository. @sravanlakkimsetti What is IBM stand on this one?

netomi commented 10 months ago

Archived repositories can indeed not be modified anymore so no commits can be pushed to them anymore. So if you plan to push some commits to them in the future you should not mark them as archived, though the archive flag can be removed at any time.

Usually a repo is archived to indicate that development has been deprecated or moved on to another repo, e.g. when moving a project to another location like Gitlab or its own GitHub organization.

merks commented 10 months ago

I think that does make it read only because I see this in the banner of such a repo:

image

So if that's bad, then we dont' want this change...

netomi commented 10 months ago

ftr: the content of these repos has already been adjusted to that:

image

akurtakov commented 10 months ago

Master branch is empty but all the R_* branches are still there is people want to backport.

netomi commented 10 months ago

Closer inspection of the repo https://github.com/eclipse-platform/eclipse.platform.text shows that commits / releases still seem to happen on a regular basis:

image

Maybe that should be reflected in the README.md of the master branch to avoid confusion, I would suggest to close this PR then.