这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@zeyugao
Copy link
Contributor

@zeyugao zeyugao commented Mar 11, 2025

Description

Fix #14659, avoid failure when unset a variable in fish

Checklist - did you ...

- [ ] Add a file to the news directory (using the template) for the next release's release notes?
- [ ] Add / update necessary tests?
- [ ] Add / update outdated documentation?

@zeyugao zeyugao requested a review from a team as a code owner March 11, 2025 15:59
@github-project-automation github-project-automation bot moved this to 🆕 New in 🔎 Review Mar 11, 2025
@conda-bot
Copy link
Contributor

We require contributors to sign our Contributor License Agreement and we don't have one on file for @zeyugao.

In order for us to review and merge your code, please e-sign the Contributor License Agreement PDF. We then need to manually verify your signature, merge the PR (conda/infrastructure#1126), and ping the bot to refresh the PR.

kenodegard
kenodegard previously approved these changes Mar 12, 2025
@github-project-automation github-project-automation bot moved this from 🆕 New to ✅ Approved in 🔎 Review Mar 12, 2025
@kenodegard
Copy link
Contributor

@conda-bot check

@conda-bot conda-bot added the cla-signed [bot] added once the contributor has signed the CLA label Mar 12, 2025
@codspeed-hq
Copy link

codspeed-hq bot commented Mar 13, 2025

CodSpeed Performance Report

Merging #14660 will not alter performance

Comparing zeyugao:main (236fb93) with main (dcc9884)

Summary

✅ 21 untouched benchmarks

@holgerdell
Copy link

I agree with this change. Either this PR or #14752 should be merged.

@scott-5
Copy link

scott-5 commented Aug 13, 2025

This problem seems to have been solved, but the code has not been merged for a long time. @kenodegard

@kenodegard kenodegard self-requested a review August 13, 2025 16:58
@kenodegard kenodegard merged commit 523e86f into conda:main Aug 13, 2025
85 checks passed
@github-project-automation github-project-automation bot moved this from ✅ Approved to 🏁 Done in 🔎 Review Aug 13, 2025
@kenodegard kenodegard mentioned this pull request Aug 13, 2025
3 tasks
@kenodegard kenodegard mentioned this pull request Sep 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed [bot] added once the contributor has signed the CLA

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

When activate in fish shell, use of set -e _CE_M; and set -e _CE_CONDA; will result in return code 4

5 participants