Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add ort-server repository #1

Merged
merged 1 commit into from
Feb 20, 2024

Conversation

mnonnenmacher
Copy link
Contributor

Add the repository for the ort-server with minimal configuration for the initial contribution. All other configuration like branch protection rules will be added after the import.

Add the repository for the ort-server with minimal configuration for the
initial contribution. All other configuration like branch protection
rules will be added after the import.

Signed-off-by: Martin Nonnenmacher <martin.nonnenmacher@bosch.com>
@mnonnenmacher mnonnenmacher requested review from a team as code owners February 18, 2024 16:01
Copy link

Diff for 13e8287:
Printing local diff:

Actions are indicated with the following symbols:
+   create
!   modify
!   forced update
-   delete

Organization technology.apoapsis[id=eclipse-apoapsis]

+   add repository[name="ort-server"] {
+     allow_auto_merge                  = true
+     allow_forking                     = true
+     allow_merge_commit                = false
+     allow_rebase_merge                = true
+     allow_squash_merge                = false
+     allow_update_branch               = true
+     archived                          = false
+     default_branch                    = "main"
+     delete_branch_on_merge            = true
+     dependabot_alerts_enabled         = true
+     dependabot_security_updates_enabled = false
+     description                       = null
+     gh_pages_build_type               = "disabled"
+     has_discussions                   = false
+     has_issues                        = true
+     has_projects                      = true
+     has_wiki                          = false
+     homepage                          = null
+     is_template                       = false
+     merge_commit_message              = "PR_TITLE"
+     merge_commit_title                = "MERGE_MESSAGE"
+     name                              = "ort-server"
+     private                           = false
+     secret_scanning                   = "enabled"
+     secret_scanning_push_protection   = "enabled"
+     squash_merge_commit_message       = "COMMIT_MESSAGES"
+     squash_merge_commit_title         = "COMMIT_OR_PR_TITLE"
+     template_repository               = null
+     topics                            = []
+     web_commit_signoff_required       = true
+   }

+   add repo_workflow_settings[repository="ort-server"] {
+     actions_can_approve_pull_request_reviews = true
+     enabled                           = true
+   }
  
  Plan: 2 to add, 0 to change, 0 to delete.
Canonical Diff for 13e8287:
Showing canonical diff:

Organization technology.apoapsis[id=eclipse-apoapsis]

--- canonical
+++ original
@@ -5,7 +5,6 @@
     orgs.newRepo('ort-server') {
       allow_auto_merge: true
       allow_squash_merge: false
-      auto_init: false
       has_wiki: false
     }
   ]

@mnonnenmacher
Copy link
Contributor Author

@netomi Can you explain why auto_init is removed in the canoncial diff? Is it because the setting is only applied once when the repository is created?

@netomi
Copy link
Contributor

netomi commented Feb 19, 2024

That looks like a Bug in the canonical view Wrt properties that are special like auto_init. You can ignore the view for now it was originally a neat idea to show the diff to a minimal config but its deprecated and will be removed when we switch your org to the App mode, when i am back from vacation on wed i will do that.

@netomi
Copy link
Contributor

netomi commented Feb 19, 2024

Here is an example how it will look like eclipse-thingweb/.eclipsefdn#15

@mnonnenmacher
Copy link
Contributor Author

That looks like a Bug in the canonical view Wrt properties that are special like auto_init. You can ignore the view for now it was originally a neat idea to show the diff to a minimal config but its deprecated and will be removed when we switch your org to the App mode, when i am back from vacation on wed i will do that.

Thanks, IIUC that means that the changes will only be automatically applied after the merge of the PR once you have enabled the app, right?

@netomi
Copy link
Contributor

netomi commented Feb 19, 2024

My Colleages can still manually apply the changes while I am away on wed I will enable the App anyways to be able to apply changes automatically after approval of a PR.

@mnonnenmacher
Copy link
Contributor Author

@mbarbero Thanks for the review, could you also merge and apply the change as mentioned by @netomi?

@heurtematte heurtematte merged commit 4c24eb7 into eclipse-apoapsis:main Feb 20, 2024
2 checks passed
@heurtematte
Copy link

New repository has been created. Committers will gain access once the sync script runs (~2h).

@netomi
Copy link
Contributor

netomi commented Feb 21, 2024

so app mode is now enabled for apoapsis as well, You can find a dashboard at https://otterdog.eclipse.org/index

it will grow over time and give you as much information about your project and its resources as possible.

@mnonnenmacher mnonnenmacher deleted the add-ort-server branch March 14, 2024 11:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants