Skip to content

Commit 8247a72

Browse files
kira-mabdelDriowya
andauthored
fix: remove deprecated class (#302)
Co-authored-by: abdelDriowya <[email protected]>
1 parent 3e700f0 commit 8247a72

File tree

4 files changed

+6
-35
lines changed

4 files changed

+6
-35
lines changed

pkg/parser/validate/executor.go

Lines changed: 2 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -34,17 +34,10 @@ func (val Validate) ValidateExecutors() {
3434
// MacOSExecutor
3535

3636
func (val Validate) validateMacOSExecutor(executor ast.MacOSExecutor) {
37-
if slices.Contains(utils.ValidXcodeAppleSiliconVersions, executor.Xcode) {
37+
if slices.Contains(utils.ValidXcodeVersions, executor.Xcode) {
3838
val.checkIfValidResourceClass(
3939
executor.ResourceClass,
40-
utils.ValidMacOSAppleSiliconResourceClasses,
41-
executor.ResourceClassRange,
42-
fmt.Sprintf("Xcode version \"%s\"", executor.Xcode),
43-
)
44-
} else if slices.Contains(utils.ValidXcodeIntelVersions, executor.Xcode) {
45-
val.checkIfValidResourceClass(
46-
executor.ResourceClass,
47-
utils.ValidMacOSIntelResourceClasses,
40+
utils.ValidMacOSResourceClasses,
4841
executor.ResourceClassRange,
4942
fmt.Sprintf("Xcode version \"%s\"", executor.Xcode),
5043
)

pkg/parser/validate/jobs_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -135,7 +135,7 @@ func TestResourceClass(t *testing.T) {
135135
yamlData: `jobs:
136136
test:
137137
macos:
138-
xcode: ` + utils.ValidXcodeAppleSiliconVersions[0] + `
138+
xcode: ` + utils.ValidXcodeVersions[0] + `
139139
resource_class: toto
140140
steps:
141141
- checkout`,

pkg/utils/executors.go

Lines changed: 3 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -184,24 +184,8 @@ var ValidMachinePairs = []struct {
184184
{Images: ValidWindowsGPUImages, ResourceClasses: ValidWindowsGPUResourceClasses},
185185
}
186186

187-
var ValidXcodeIntelVersions = []string{
188-
"15.3.0",
189-
"15.2.0",
190-
"15.1.0",
191-
"15.0.0",
192-
"14.3.1",
193-
"14.2.0",
194-
"14.1.0",
195-
"14.0.1",
196-
"13.4.1",
197-
"12.5.1",
198-
}
199-
200-
var ValidMacOSIntelResourceClasses = []string{
201-
"macos.x86.medium.gen2",
202-
}
203-
204-
var ValidXcodeAppleSiliconVersions = []string{
187+
var ValidXcodeVersions = []string{
188+
"16.0.0",
205189
"15.4.0",
206190
"15.3.0",
207191
"15.2.0",
@@ -214,16 +198,11 @@ var ValidXcodeAppleSiliconVersions = []string{
214198
"13.4.1",
215199
}
216200

217-
var ValidMacOSAppleSiliconResourceClasses = []string{
201+
var ValidMacOSResourceClasses = []string{
218202
"macos.m1.medium.gen1",
219203
"macos.m1.large.gen1",
220204
}
221205

222-
var ValidMacOSResourceClasses = slices.Concat(
223-
ValidMacOSAppleSiliconResourceClasses,
224-
ValidMacOSIntelResourceClasses,
225-
)
226-
227206
var ValidDockerResourceClasses = ValidLinuxResourceClasses
228207

229208
func IsSelfHostedRunner(resourceClass string) bool {

publicschema.json

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -517,7 +517,6 @@
517517
"gpu.nvidia.small",
518518
"gpu.nvidia.medium",
519519
"windows.gpu.nvidia.medium",
520-
"macos.x86.medium.gen2",
521520
"macos.m1.medium.gen1",
522521
"macos.m1.large.gen1"
523522
]

0 commit comments

Comments
 (0)