Skip to content

Commit 7f1729a

Browse files
hramosfacebook-github-bot
authored andcommitted
Circle CI: Replace string parameters with enums where applicable (#34962)
Summary: Pull Request resolved: #34962 For jobs that take a string parameter, where that string parameter is expected to be one of a set of accepted values, use an enum instead. If an unexpected value is passed to a enum parameter, then `circleci config validate` will flag the issue. Changelog: [Internal] Reviewed By: NickGerleman Differential Revision: D40310118 fbshipit-source-id: b8416c415705ff6eba80cc5f0d9bfe670569f732
1 parent 3d64d09 commit 7f1729a

1 file changed

Lines changed: 23 additions & 9 deletions

File tree

.circleci/config.yml

Lines changed: 23 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -744,14 +744,18 @@ jobs:
744744
executor: reactnativeandroid
745745
parameters:
746746
flavor:
747-
type: string
748747
default: "Debug"
748+
description: The Android build type. Must be one of "Debug", "Release".
749+
type: enum
750+
enum: ["Debug", "Release"]
749751
newarchitecture:
750752
type: boolean
751753
default: false
752754
jsengine:
753-
type: string
754755
default: "Hermes"
756+
description: Which JavaScript engine to use. Must be one of "Hermes", "JSC".
757+
type: enum
758+
enum: ["Hermes", "JSC"]
755759
environment:
756760
- PROJECT_NAME: "AndroidTemplateProject"
757761
steps:
@@ -790,17 +794,25 @@ jobs:
790794
executor: reactnativeios
791795
parameters:
792796
flavor:
793-
type: string
794797
default: "Debug"
798+
description: The Xcode build type. Must be one of "Debug", "Release".
799+
type: enum
800+
enum: ["Debug", "Release"]
795801
architecture:
796-
type: string
797802
default: "OldArch"
803+
description: Which React Native architecture to use. Must be one of "NewArch", "OldArch".
804+
type: enum
805+
enum: ["NewArch", "OldArch"]
798806
jsengine:
799-
type: string
800807
default: "Hermes"
808+
description: Which JavaScript engine to use. Must be one of "Hermes", "JSC".
809+
type: enum
810+
enum: ["Hermes", "JSC"]
801811
flipper:
802-
type: string
803812
default: "WithFlipper"
813+
description: Whether Flipper is enabled. Must be one of "WithFlipper", "WithoutFlipper".
814+
type: enum
815+
enum: ["WithFlipper", "WithoutFlipper"]
804816
environment:
805817
- PROJECT_NAME: "iOSTemplateProject"
806818
- HERMES_WS_DIR: *hermes_workspace_root
@@ -878,14 +890,16 @@ jobs:
878890
test_ios_rntester:
879891
executor: reactnativeios
880892
parameters:
881-
architecture:
882-
type: string
883-
default: "OldArch"
884893
jsengine:
885894
default: "Hermes"
886895
description: Which JavaScript engine to use. Must be one of "Hermes", "JSC".
887896
type: enum
888897
enum: ["Hermes", "JSC"]
898+
architecture:
899+
default: "OldArch"
900+
description: Which React Native architecture to use. Must be one of "OldArch", "NewArch".
901+
type: enum
902+
enum: ["NewArch", "OldArch"]
889903
steps:
890904
- checkout_code_with_cache
891905
- run_yarn

0 commit comments

Comments
 (0)