Skip to content

Workflow refactoring and improvements #273

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

jeffhandley
Copy link
Collaborator

Refactor and improve the GitHub workflows.

  1. Remove the step of code coverage that posts sticky comments to PRs. We might bring this back later, but we don't need it for now.
  2. Only run the markdown-link-check job if markdown files are updated.
  3. Allow the Build and Test workflow to be triggered manually, to run on every push to main, and to run on PRs that touch code, regardless of the target branch.
  4. Declare all workflow permissions explicitly, keeping them as minimal as possible.
  5. Refactor the release workflow for the 3 different package publishing jobs to run in parallel (and prevent one of them failing from preventing the others from running).
  6. Pin all actions to specific SHAs, aligning with GitHub's recommended best practices.

@jeffhandley jeffhandley added the area-infrastructure Related to repo infrastructure label Apr 10, 2025
@jeffhandley jeffhandley self-assigned this Apr 10, 2025
Copy link
Contributor

@Copilot 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.

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

@jeffhandley jeffhandley requested a review from Copilot April 11, 2025 00:22
Copy link
Contributor

@Copilot 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.

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

@jeffhandley jeffhandley merged commit f1cc2d8 into modelcontextprotocol:main Apr 11, 2025
13 of 14 checks passed
@jeffhandley jeffhandley deleted the jeffhandley/workflows branch April 11, 2025 19:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-infrastructure Related to repo infrastructure
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants