Skip to content

Commit 26dfde7

Browse files
authored
Merge pull request #337 from Unity-Technologies/ci-cleanups
Further integration of wrench
2 parents aa61e6c + 18f9ad0 commit 26dfde7

File tree

9 files changed

+29
-76
lines changed

9 files changed

+29
-76
lines changed

.gitignore

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44
[Tt]emp/
55
[Ll]ogs/
66
DockerFile
7+
Tools/CI/bin/*
78
.vs
89
.vscode
910
.DS_Store
@@ -16,4 +17,4 @@ android-sdk/*
1617
*.csproj
1718
*.suo
1819
*.sublime-project
19-
*.sublime-workspace
20+
*.sublime-workspace

.yamato/format.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ check_format:
66
flavor: b1.small
77
commands:
88
- git clone --branch stable [email protected]:unity/unity-meta.git --no-checkout
9-
- git -C unity-meta checkout 6f6dd71b42f7fd3294032a6c89b1402ef3e46099
9+
- git -C unity-meta checkout 05b2852f2e25f0b7155756c870f870b932c97ae9
1010
- perl unity-meta/Tools/Format/format.pl --dry-run ./com.unity.mobile.notifications
1111
triggers:
1212
branches:

.yamato/upm-ci.yml

Lines changed: 1 addition & 55 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
test_editors:
22
- version: trunk
3+
- version: 6000.2
34
- version: 6000.1
45
- version: 6000.0
56
- version: 2022.3
@@ -20,29 +21,6 @@ utr:
2021

2122
---
2223

23-
{% for editor in test_editors %}
24-
{% for platform in test_platforms %}
25-
test_{{ platform.name }}_{{ editor.version }}:
26-
name : Test {{ editor.version }} on {{ platform.name }}
27-
agent:
28-
type: {{ platform.type }}
29-
image: {{ platform.image }}
30-
flavor: {{ platform.flavor}}
31-
commands:
32-
- npm install upm-ci-utils@stable -g --registry https://artifactory.prd.cds.internal.unity3d.com/artifactory/api/npm/upm-npm
33-
- upm-ci package test --package-path com.unity.mobile.notifications --unity-version {{ editor.version }} --platform editmode
34-
artifacts:
35-
logs:
36-
paths:
37-
- "upm-ci~/test-results/**/*"
38-
packages:
39-
paths:
40-
- "upm-ci~/packages/**/*"
41-
dependencies:
42-
- .yamato/wrench/package-pack-jobs.yml#package_pack_-_mobile_notifications
43-
{% endfor %}
44-
{% endfor %}
45-
4624
{% for editor in test_editors %}
4725
test_Android_{{ editor.version }}:
4826
name: Test {{ editor.version }} on Android
@@ -116,38 +94,6 @@ test_trigger:
11694
dependencies:
11795
- .yamato/wrench/package-pack-jobs.yml#package_pack_-_mobile_notifications
11896
{% for editor in test_editors %}
119-
{% for platform in test_platforms %}
120-
- .yamato/upm-ci.yml#test_{{ platform.name }}_{{ editor.version }}
121-
{% endfor %}
122-
{% endfor %}
123-
{% for editor in test_editors %}
12497
- .yamato/upm-ci.yml#test_Android_{{ editor.version }}
12598
- .yamato/upm-ci.yml#test_iOS_{{ editor.version }}
12699
{% endfor %}
127-
128-
publish:
129-
name: Publish to Internal Registry
130-
agent:
131-
type: Unity::VM
132-
image: package-ci/win10:stable
133-
flavor: b1.large
134-
variables:
135-
UPMCI_ENABLE_PACKAGE_SIGNING: 1
136-
commands:
137-
- npm install upm-ci-utils@stable -g --registry https://artifactory.prd.cds.internal.unity3d.com/artifactory/api/npm/upm-npm
138-
- upm-ci package publish --package-path com.unity.mobile.notifications
139-
triggers:
140-
tags:
141-
only:
142-
- /^(r|R)(c|C)-\d+\.\d+\.\d+(-preview(\.\d+)?)?$/
143-
artifacts:
144-
artifacts:
145-
paths:
146-
- "upm-ci~/packages/*.tgz"
147-
dependencies:
148-
- .yamato/wrench/package-pack-jobs.yml#package_pack_-_mobile_notifications
149-
{% for editor in test_editors %}
150-
{% for platform in test_platforms %}
151-
- .yamato/upm-ci.yml#test_{{ platform.name }}_{{ editor.version }}
152-
{% endfor %}
153-
{% endfor %}

.yamato/wrench/promotion-jobs.yml

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,17 @@
11
# Auto-generated by Recipe Engine, do not modify manually.
22
# This job is generated by the wrench recipe engine module, see find the docs here: http://Go/ii2fb
33

4+
# Trigger for mobile.notifications Custom Checks
5+
mobile_notifications_custom_checks:
6+
name: mobile.notifications Custom Checks
7+
dependencies:
8+
- path: .yamato/upm-ci.yml#test_trigger
9+
triggers:
10+
expression: push.branch match "^release/.*"
11+
cancel_old_ci: true
12+
metadata:
13+
Job Maintainers: '#release-management'
14+
415
# Publish Dry Run for mobile.notifications to https://artifactory-slo.bf.unity3d.com/artifactory/api/npm/upm-npm
516
publish_dry_run_mobile_notifications:
617
name: Publish Dry Run mobile.notifications

.yamato/wrench/recipe-regeneration.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,14 +9,14 @@ test_-_wrench_jobs_up_to_date:
99
type: Unity::VM
1010
flavor: b1.large
1111
commands:
12-
- command: dotnet run --project \Tools\CI\UnityMobileNotifications.Cookbook.csproj
12+
- command: dotnet run --project Tools/CI/UnityMobileNotifications.Cookbook.csproj
1313
- command: |-
1414
if [ -n "$(git status --porcelain)" ]; then
1515
git status
1616
echo "Your repo is not clean - diff output:"
1717
git diff
1818
echo "You must run recipe generation after updating recipes to update the generated YAML!"
19-
echo "Run 'dotnet run --project \Tools\CI\UnityMobileNotifications.Cookbook.csproj' from the root of your repository to regenerate all job definitions created by wrench."
19+
echo "Run 'dotnet run --project Tools/CI/UnityMobileNotifications.Cookbook.csproj' from the root of your repository to regenerate all job definitions created by wrench."
2020
exit 1
2121
fi
2222
variables:

.yamato/wrench/wrench_config.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,12 +26,13 @@
2626
"com.unity.mobile.notifications": [
2727
".yamato/wrench/api-validation-jobs.yml#api_validation_-_mobile_notifications_-_2021_3_-_windows",
2828
".yamato/wrench/preview-a-p-v.yml#all_preview_apv_jobs",
29+
".yamato/wrench/promotion-jobs.yml#mobile_notifications_custom_checks",
2930
".yamato/wrench/promotion-jobs.yml#publish_dry_run_mobile_notifications"
3031
]
3132
},
3233
"publishing_job": ".yamato/wrench/promotion-jobs.yml#publish_mobile_notifications",
3334
"branch_pattern": "ReleaseSlash",
3435
"wrench_version": "0.10.32.0",
3536
"pvp_exemption_path": ".yamato/wrench/pvp-exemptions.json",
36-
"cs_project_path": "\\Tools\\CI\\UnityMobileNotifications.Cookbook.csproj"
37+
"cs_project_path": "Tools/CI/UnityMobileNotifications.Cookbook.csproj"
3738
}

.yamato/xray-exemptions.json

Lines changed: 0 additions & 13 deletions
This file was deleted.

Tools/CI/Settings/UnityMobileNotificationsSettings.cs

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
1-
using RecipeEngine.Api.Settings;
1+
using RecipeEngine.Api.Dependencies;
2+
using RecipeEngine.Api.Settings;
23
using RecipeEngine.Modules.Wrench.Models;
34
using RecipeEngine.Modules.Wrench.Settings;
45

@@ -14,7 +15,11 @@ public class UnityMobileNotificationsSettings : AnnotatedSettingsBase
1415
{
1516
{
1617
"com.unity.mobile.notifications",
17-
new PackageOptions() { ReleaseOptions = new ReleaseOptions() { IsReleasing = true } }
18+
new PackageOptions()
19+
{
20+
ReleaseOptions = new ReleaseOptions() { IsReleasing = true },
21+
CustomChecks = new HashSet<Dependency>() { new Dependency("upm-ci", "test_trigger") }
22+
}
1823
}
1924
};
2025

Tools/CI/regenerate.sh

100644100755
Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,4 @@
1+
#!/bin/sh
2+
13
cd $(dirname "$0")/../../
2-
dotnet run --project Tools\CI\UnityMobileNotifications.Cookbook.csproj
4+
dotnet run --project Tools/CI/UnityMobileNotifications.Cookbook.csproj

0 commit comments

Comments
 (0)