Skip to content

Conversation

@saicheranb
Copy link

@saicheranb saicheranb commented Aug 13, 2025

DEVOPS-820 - use shared github workflows in geoapps

@github-actions github-actions bot changed the title DEVOPS-820 : use shared github workflows in geoapps DEVOPS-820: use shared github workflows in geoapps Aug 13, 2025
@saicheranb saicheranb requested a review from sebhmg August 14, 2025 01:40
@sebhmg sebhmg changed the base branch from main to develop August 14, 2025 02:43
sebhmg
sebhmg previously approved these changes Aug 14, 2025
@sebhmg sebhmg requested a review from Copilot August 14, 2025 03:01

This comment was marked as outdated.

@sebhmg sebhmg requested a review from Copilot August 14, 2025 05:01
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 migrates GitHub workflows to use shared workflows from the MiraGeoscience/CI-tools repository. The changes standardize workflow configurations across geoapps by leveraging centralized, reusable workflow templates.

  • Adds a new Zizmor security analysis workflow using shared templates
  • Updates existing workflows to use explicit permissions and remove deprecated configurations
  • Replaces secrets: inherit with explicit secret passing for improved security

Reviewed Changes

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

File Description
.github/workflows/zizmor-security.yml New security analysis workflow using shared Zizmor template
.github/workflows/python_analysis.yml Updated to use explicit permissions and reorganized LFS configuration
.github/workflows/pr_add_jira_summary.yml Migrated from pull_request_target to pull_request with explicit permissions and secrets
.github/workflows/issue_to_jira.yml Added explicit permissions and replaced inherited secrets with specific ones

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@sebhmg sebhmg merged commit da8ccf6 into develop Aug 18, 2025
8 of 10 checks passed
@sebhmg sebhmg deleted the DEVOPS-820 branch August 18, 2025 19:47
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.

3 participants