Skip to content

Commit d075a56

Browse files
authored
Fix merge conflict (#41853)
Closes #ISSUE Release Notes: - N/A
1 parent 8217e57 commit d075a56

File tree

8 files changed

+27
-27
lines changed

8 files changed

+27
-27
lines changed

.github/workflows/release.yml

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -63,6 +63,10 @@ jobs:
6363
- name: steps::install_mold
6464
run: ./script/install-mold
6565
shell: bash -euxo pipefail {0}
66+
- name: steps::cache_rust_dependencies_namespace
67+
uses: namespacelabs/nscloud-cache-action@v1
68+
with:
69+
cache: rust
6670
- name: steps::setup_node
6771
uses: actions/setup-node@49933ea5288caeca8642d1e84afbd3f7d6820020
6872
with:
@@ -74,7 +78,7 @@ jobs:
7478
run: cargo install cargo-nextest --locked
7579
shell: bash -euxo pipefail {0}
7680
- name: steps::clear_target_dir_if_large
77-
run: ./script/clear-target-dir-if-larger-than 100
81+
run: ./script/clear-target-dir-if-larger-than 250
7882
shell: bash -euxo pipefail {0}
7983
- name: steps::cargo_nextest
8084
run: cargo nextest run --workspace --no-fail-fast --failure-output immediate-final

.github/workflows/run_agent_evals.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -30,10 +30,10 @@ jobs:
3030
uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683
3131
with:
3232
clean: false
33-
- name: steps::cache_rust_dependencies
34-
uses: swatinem/rust-cache@9d47c6ad4b02e050fd481d890b2ea34778fd09d6
33+
- name: steps::cache_rust_dependencies_namespace
34+
uses: namespacelabs/nscloud-cache-action@v1
3535
with:
36-
save-if: ${{ github.ref == 'refs/heads/main' }}
36+
cache: rust
3737
- name: steps::setup_linux
3838
run: ./script/linux
3939
shell: bash -euxo pipefail {0}

.github/workflows/run_tests.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -164,7 +164,7 @@ jobs:
164164
run: cargo install cargo-nextest --locked
165165
shell: bash -euxo pipefail {0}
166166
- name: steps::clear_target_dir_if_large
167-
run: ./script/clear-target-dir-if-larger-than 100
167+
run: ./script/clear-target-dir-if-larger-than 250
168168
shell: bash -euxo pipefail {0}
169169
- name: steps::cargo_nextest
170170
run: cargo nextest run --workspace --no-fail-fast --failure-output immediate-final

.github/workflows/run_unit_evals.yml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -23,10 +23,10 @@ jobs:
2323
mkdir -p ./../.cargo
2424
cp ./.cargo/ci-config.toml ./../.cargo/config.toml
2525
shell: bash -euxo pipefail {0}
26-
- name: steps::cache_rust_dependencies
27-
uses: swatinem/rust-cache@9d47c6ad4b02e050fd481d890b2ea34778fd09d6
26+
- name: steps::cache_rust_dependencies_namespace
27+
uses: namespacelabs/nscloud-cache-action@v1
2828
with:
29-
save-if: ${{ github.ref == 'refs/heads/main' }}
29+
cache: rust
3030
- name: steps::setup_linux
3131
run: ./script/linux
3232
shell: bash -euxo pipefail {0}
@@ -37,7 +37,7 @@ jobs:
3737
run: cargo install cargo-nextest --locked
3838
shell: bash -euxo pipefail {0}
3939
- name: steps::clear_target_dir_if_large
40-
run: ./script/clear-target-dir-if-larger-than 100
40+
run: ./script/clear-target-dir-if-larger-than 250
4141
shell: bash -euxo pipefail {0}
4242
- name: ./script/run-unit-evals
4343
run: ./script/run-unit-evals

tooling/xtask/src/tasks/workflows/release.rs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,7 @@ pub(crate) fn create_sentry_release() -> Step<Use> {
9090
)
9191
.add_env(("SENTRY_ORG", "zed-dev"))
9292
.add_env(("SENTRY_PROJECT", "zed"))
93-
.add_env(("SENTRY_AUTH_TOKEN", "${{ secrets.SENTRY_AUTH_TOKEN }}"))
93+
.add_env(("SENTRY_AUTH_TOKEN", vars::SENTRY_AUTH_TOKEN))
9494
.add_with(("environment", "production"))
9595
}
9696

@@ -109,7 +109,7 @@ fn auto_release_preview(deps: &[&NamedJob; 1]) -> NamedJob {
109109
steps::script(
110110
r#"gh release edit "$GITHUB_REF_NAME" --repo=zed-industries/zed --draft=false"#,
111111
)
112-
.add_env(("GITHUB_TOKEN", "${{ secrets.GITHUB_TOKEN }}")),
112+
.add_env(("GITHUB_TOKEN", vars::GITHUB_TOKEN)),
113113
)
114114
.add_step(create_sentry_release()),
115115
)
@@ -146,7 +146,7 @@ fn upload_release_assets(deps: &[&NamedJob], bundle: &ReleaseBundleJobs) -> Name
146146
.add_step(prep_release_artifacts())
147147
.add_step(
148148
steps::script("gh release upload \"$GITHUB_REF_NAME\" --repo=zed-industries/zed release-artifacts/*")
149-
.add_env(("GITHUB_TOKEN", "${{ secrets.GITHUB_TOKEN }}")),
149+
.add_env(("GITHUB_TOKEN", vars::GITHUB_TOKEN)),
150150
),
151151
)
152152
}
@@ -160,7 +160,7 @@ fn create_draft_release() -> NamedJob {
160160

161161
fn create_release() -> Step<Run> {
162162
named::bash("script/create-draft-release target/release-notes.md")
163-
.add_env(("GITHUB_TOKEN", "${{ secrets.GITHUB_TOKEN }}"))
163+
.add_env(("GITHUB_TOKEN", vars::GITHUB_TOKEN))
164164
}
165165

166166
named::job(

tooling/xtask/src/tasks/workflows/run_agent_evals.rs

Lines changed: 7 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -25,11 +25,8 @@ pub(crate) fn run_agent_evals() -> Workflow {
2525
.add_env(("CARGO_TERM_COLOR", "always"))
2626
.add_env(("CARGO_INCREMENTAL", 0))
2727
.add_env(("RUST_BACKTRACE", 1))
28-
.add_env(("ANTHROPIC_API_KEY", "${{ secrets.ANTHROPIC_API_KEY }}"))
29-
.add_env((
30-
"ZED_CLIENT_CHECKSUM_SEED",
31-
"${{ secrets.ZED_CLIENT_CHECKSUM_SEED }}",
32-
))
28+
.add_env(("ANTHROPIC_API_KEY", vars::ANTHROPIC_API_KEY))
29+
.add_env(("ZED_CLIENT_CHECKSUM_SEED", vars::ZED_CLIENT_CHECKSUM_SEED))
3330
.add_env(("ZED_EVAL_TELEMETRY", 1))
3431
.add_job(agent_evals.name, agent_evals.job)
3532
}
@@ -48,7 +45,7 @@ fn agent_evals() -> NamedJob {
4845
.runs_on(runners::LINUX_DEFAULT)
4946
.timeout_minutes(60_u32)
5047
.add_step(steps::checkout_repo())
51-
.add_step(steps::cache_rust_dependencies())
48+
.add_step(steps::cache_rust_dependencies_namespace())
5249
.map(steps::install_linux_dependencies)
5350
.add_step(setup_cargo_config(Platform::Linux))
5451
.add_step(steps::script("cargo build --package=eval"))
@@ -71,10 +68,7 @@ pub(crate) fn run_unit_evals() -> Workflow {
7168
.add_env(("CARGO_TERM_COLOR", "always"))
7269
.add_env(("CARGO_INCREMENTAL", 0))
7370
.add_env(("RUST_BACKTRACE", 1))
74-
.add_env((
75-
"ZED_CLIENT_CHECKSUM_SEED",
76-
"${{ secrets.ZED_CLIENT_CHECKSUM_SEED }}",
77-
))
71+
.add_env(("ZED_CLIENT_CHECKSUM_SEED", vars::ZED_CLIENT_CHECKSUM_SEED))
7872
.add_job(unit_evals.name, unit_evals.job)
7973
}
8074

@@ -87,7 +81,7 @@ fn unit_evals() -> NamedJob {
8781
)
8882
.if_condition(Expression::new("${{ failure() }}"))
8983
.add_with(("method", "chat.postMessage"))
90-
.add_with(("token", "${{ secrets.SLACK_APP_ZED_UNIT_EVALS_BOT_TOKEN }}"))
84+
.add_with(("token", vars::SLACK_APP_ZED_UNIT_EVALS_BOT_TOKEN))
9185
.add_with(("payload", indoc::indoc!{r#"
9286
channel: C04UDRNNJFQ
9387
text: "Unit Evals Failed: https://github.com/zed-industries/zed/actions/runs/${{ github.run_id }}"
@@ -99,13 +93,13 @@ fn unit_evals() -> NamedJob {
9993
.runs_on(runners::LINUX_DEFAULT)
10094
.add_step(steps::checkout_repo())
10195
.add_step(steps::setup_cargo_config(Platform::Linux))
102-
.add_step(steps::cache_rust_dependencies())
96+
.add_step(steps::cache_rust_dependencies_namespace())
10397
.map(steps::install_linux_dependencies)
10498
.add_step(steps::cargo_install_nextest(Platform::Linux))
10599
.add_step(steps::clear_target_dir_if_large(Platform::Linux))
106100
.add_step(
107101
steps::script("./script/run-unit-evals")
108-
.add_env(("ANTHROPIC_API_KEY", "${{ secrets.ANTHROPIC_API_KEY }}")),
102+
.add_env(("ANTHROPIC_API_KEY", vars::ANTHROPIC_API_KEY)),
109103
)
110104
.add_step(send_failure_to_slack())
111105
.add_step(steps::cleanup_cargo_config(Platform::Linux)),

tooling/xtask/src/tasks/workflows/steps.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,7 @@ pub fn cleanup_cargo_config(platform: Platform) -> Step<Run> {
8989
pub fn clear_target_dir_if_large(platform: Platform) -> Step<Run> {
9090
match platform {
9191
Platform::Windows => named::pwsh("./script/clear-target-dir-if-larger-than.ps1 250"),
92-
Platform::Linux => named::bash("./script/clear-target-dir-if-larger-than 100"),
92+
Platform::Linux => named::bash("./script/clear-target-dir-if-larger-than 250"),
9393
Platform::Mac => named::bash("./script/clear-target-dir-if-larger-than 300"),
9494
}
9595
}

tooling/xtask/src/tasks/workflows/vars.rs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@ macro_rules! var {
1616
};
1717
}
1818

19+
secret!(ANTHROPIC_API_KEY);
1920
secret!(APPLE_NOTARIZATION_ISSUER_ID);
2021
secret!(APPLE_NOTARIZATION_KEY);
2122
secret!(APPLE_NOTARIZATION_KEY_ID);
@@ -32,6 +33,7 @@ secret!(SENTRY_AUTH_TOKEN);
3233
secret!(ZED_CLIENT_CHECKSUM_SEED);
3334
secret!(ZED_CLOUD_PROVIDER_ADDITIONAL_MODELS_JSON);
3435
secret!(ZED_SENTRY_MINIDUMP_ENDPOINT);
36+
secret!(SLACK_APP_ZED_UNIT_EVALS_BOT_TOKEN);
3537

3638
// todo(ci) make these secrets too...
3739
var!(AZURE_SIGNING_ACCOUNT_NAME);

0 commit comments

Comments
 (0)