Skip to content

Make update-conda.py executable#41585

Merged
vbraun merged 1 commit intosagemath:developfrom
saraedum:update-conda-x
Feb 11, 2026
Merged

Make update-conda.py executable#41585
vbraun merged 1 commit intosagemath:developfrom
saraedum:update-conda-x

Conversation

@saraedum
Copy link
Copy Markdown
Member

@saraedum saraedum commented Feb 3, 2026

so the command from the tools/README.md works

Within an active virtual environment where grayskull and conda-lock are installed, run the following command:

tools/update-conda.py

📝 Checklist

  • The title is concise and informative.
  • The description explains in detail what this PR is about.
  • I have linked a relevant issue or discussion. No issue for this. Just something I found while trying to update the environment files.
  • I have created tests covering the changes. No, I think that would go too far.
  • I have updated the documentation and checked the documentation preview. No, I updated the facts to match the docs.

⌛ Dependencies

so the command from the tools/README.md works

> Within an active virtual environment where `grayskull` and `conda-lock` are installed, run the following command:

> tools/update-conda.py
@github-actions
Copy link
Copy Markdown

github-actions bot commented Feb 3, 2026

Documentation preview for this PR (built with commit 8ada9bf; changes) is ready! 🎉
This preview will update shortly after each push to this PR.

Copy link
Copy Markdown
Contributor

@tobiasdiez tobiasdiez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

vbraun pushed a commit to vbraun/sage that referenced this pull request Feb 5, 2026
sagemathgh-41585: Make update-conda.py executable
    
so the command from the tools/README.md works

> Within an active virtual environment where `grayskull` and `conda-
lock` are installed, run the following command:

> tools/update-conda.py

### 📝 Checklist

<!-- Put an `x` in all the boxes that apply. -->

- [x] The title is concise and informative.
- [x] The description explains in detail what this PR is about.
- [x] I have linked a relevant issue or discussion. No issue for this.
Just something I found while trying to update the environment files.
- [x] I have created tests covering the changes. No, I think that would
go too far.
- [x] I have updated the documentation and checked the documentation
preview. No, I updated the facts to match the docs.

### ⌛ Dependencies

<!-- List all open PRs that this PR logically depends on. For example,
-->
<!-- - sagemath#12345: short description why this is a dependency -->
<!-- - sagemath#34567: ... -->
    
URL: sagemath#41585
Reported by: Julian Rüth
Reviewer(s): Tobias Diez
vbraun pushed a commit to vbraun/sage that referenced this pull request Feb 7, 2026
sagemathgh-41585: Make update-conda.py executable
    
so the command from the tools/README.md works

> Within an active virtual environment where `grayskull` and `conda-
lock` are installed, run the following command:

> tools/update-conda.py

### 📝 Checklist

<!-- Put an `x` in all the boxes that apply. -->

- [x] The title is concise and informative.
- [x] The description explains in detail what this PR is about.
- [x] I have linked a relevant issue or discussion. No issue for this.
Just something I found while trying to update the environment files.
- [x] I have created tests covering the changes. No, I think that would
go too far.
- [x] I have updated the documentation and checked the documentation
preview. No, I updated the facts to match the docs.

### ⌛ Dependencies

<!-- List all open PRs that this PR logically depends on. For example,
-->
<!-- - sagemath#12345: short description why this is a dependency -->
<!-- - sagemath#34567: ... -->
    
URL: sagemath#41585
Reported by: Julian Rüth
Reviewer(s): Tobias Diez
@vbraun vbraun merged commit 44d9603 into sagemath:develop Feb 11, 2026
17 of 22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants