Skip to content

Devops updates [DO NOT MERGE]#19

Open
heckerdj wants to merge 6 commits intomainfrom
devops_updates
Open

Devops updates [DO NOT MERGE]#19
heckerdj wants to merge 6 commits intomainfrom
devops_updates

Conversation

@heckerdj
Copy link
Copy Markdown
Owner

@heckerdj heckerdj commented Oct 6, 2025

To merge when preview option is released: https://github.com/actions/deploy-pages?tab=readme-ov-file#inputs-

@heckerdj heckerdj requested a review from Copilot October 6, 2025 03:54
Copy link
Copy Markdown
Contributor

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 consolidates GitHub Actions workflows and includes some CSS cleanup. The main purpose is to streamline DevOps processes by merging PR preview functionality into the main build workflow and removing duplicate configurations.

Key changes:

  • Consolidated PR preview workflow into the main build-and-deploy workflow
  • Removed duplicate CSS properties and cleaned up formatting
  • Simplified SonarCloud workflow formatting

Reviewed Changes

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

File Description
src/components/Tabs.css Removed duplicate CSS properties and cleaned up transition declarations
.github/workflows/sonarcloud.yml Cleaned up comment formatting by removing empty comment lines
.github/workflows/pr-preview.yml Completely removed this file as functionality was moved to build-and-deploy.yml
.github/workflows/build-and-deploy.yml Enhanced to handle both production deploys and PR previews with conditional logic

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

@heckerdj heckerdj changed the title Devops updates Devops updates [DO NOT MERGE] Oct 6, 2025
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.

2 participants