Skip to content

Updated Workflow files#383

Merged
Castro19 merged 1 commit intomainfrom
workflow-files
Jun 30, 2025
Merged

Updated Workflow files#383
Castro19 merged 1 commit intomainfrom
workflow-files

Conversation

@Castro19
Copy link
Copy Markdown
Owner

📌 Summary

🔍 Related Issues

Closes #

🛠 Changes Made

✅ Checklist

  • My code follows the PolyLink Contribution Guidelines.
  • I have tested my changes to ensure they work as expected.
  • I have documented my changes (if applicable).
  • My PR has a clear title and description.

📸 Screenshots (if applicable)

❓ Additional Notes

@Castro19 Castro19 requested a review from Copilot June 30, 2025 17:37
@Castro19 Castro19 self-assigned this Jun 30, 2025
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 streamlines the CI/CD setup by removing obsolete workflows and consolidating deployment steps into the main pipeline.

  • Deleted two redundant workflow files (main_polylinkapp.yml and the static-web-apps workflow)
  • Updated Azure CLI credentials, app name, and package settings in main_polylink.yml

Reviewed Changes

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

File Description
.github/workflows/main_polylinkapp.yml Removed redundant Node.js Azure Web App workflow
.github/workflows/main_polylink.yml Updated Azure login secrets, renamed app, and adjusted package
.github/workflows/azure-static-web-apps-yellow-cliff-0ca9c001e.yml Removed static web apps CI/CD workflow
Comments suppressed due to low confidence (4)

.github/workflows/main_polylink.yml:104

  • [nitpick] Inconsistent quoting style for step names; consider using a consistent approach (either quoted or unquoted) across all steps to improve readability.
      - name: "Deploy to Azure Web App"

.github/workflows/main_polylink.yml:110

  • Removing the type: runFromPackage option and deploying the entire directory may slow down deployments; consider re-adding type: runFromPackage or packaging a ZIP to optimize deployment performance.
          package: .

@Castro19 Castro19 merged commit ad10d46 into main Jun 30, 2025
10 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.

2 participants