Skip to content

ci: upgrade and simplify#9

Merged
sellisd merged 1 commit intomainfrom
DSE/ci
Nov 4, 2025
Merged

ci: upgrade and simplify#9
sellisd merged 1 commit intomainfrom
DSE/ci

Conversation

@sellisd
Copy link
Collaborator

@sellisd sellisd commented Nov 4, 2025

No description provided.

@sellisd sellisd requested a review from Copilot November 4, 2025 09:20
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the CI workflows to modernize the uv setup, adds security permissions, and adjusts the trigger configuration. The changes simplify Python environment setup by using the official astral-sh/setup-uv action and configures workflows to run only on pull requests instead of both push and PR events.

  • Simplified uv installation by replacing manual curl-based installation with the official astral-sh/setup-uv action
  • Added explicit contents: read permissions at the workflow level for better security
  • Updated Dependabot configuration to track uv ecosystem instead of pip

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
.github/workflows/ci.yml Removed matrix testing for multiple Python versions, switched to uv-managed Python installation, added permissions, and changed trigger to pull_request only
.github/workflows/badge-wall.yml Updated to use official uv setup action, added permissions, and changed trigger from push to pull_request
.github/dependabot.yml Changed package ecosystem from "pip" to "uv" for dependency tracking

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@sellisd sellisd merged commit 1545a51 into main Nov 4, 2025
8 checks passed
@sellisd sellisd deleted the DSE/ci branch November 4, 2025 09:23
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.

1 participant