Skip to content

Update workflows#348

Merged
Just-Insane merged 2 commits into
integrationfrom
workflow-updates
Jun 23, 2026
Merged

Update workflows#348
Just-Insane merged 2 commits into
integrationfrom
workflow-updates

Conversation

@Just-Insane

Copy link
Copy Markdown
Collaborator

Description

Tweaks the workflows to not deploy sentry toolbar to production

Fixes #

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings

AI disclosure:

  • Partially AI assisted (clarify which code was AI assisted and briefly explain what it does).
  • Fully AI generated (explain what all the generated code does in moderate detail).

@Just-Insane Just-Insane enabled auto-merge June 23, 2026 23:04
@Just-Insane Just-Insane changed the title tUpdate workflows Update workflows Jun 23, 2026
@github-actions

github-actions Bot commented Jun 23, 2026

Copy link
Copy Markdown

OpenTofu plan for production

Plan: 3 to add, 0 to change, 3 to destroy.
OpenTofu used the selected providers to generate the following execution
plan. Resource actions are indicated with the following symbols:
-/+ destroy and then create replacement

OpenTofu will perform the following actions:

  # cloudflare_worker_version.site must be replaced
-/+ resource "cloudflare_worker_version" "site" {
!~      annotations         = {
+           workers_message      = (known after apply)
+           workers_tag          = (known after apply)
!~          workers_triggered_by = "create_version_api" -> (known after apply)
        } -> (known after apply)
!~      assets              = { # forces replacement
!~          asset_manifest_sha256 = "cfb29faf79c62cc3ab25e96e1f8c626997964acf0104c8d1c40e1161c4b80710" -> "d17c364976a9d0551091a0a0ad4869fac8fe19079cd0ca78c9cf8adbe391648b" # forces replacement
!~          directory             = "/home/runner/work/Charm/Charm/dist/client" -> "/github/workspace/dist/client"
#            (1 unchanged attribute hidden)
        }
+       compatibility_flags = (known after apply)
!~      created_on          = "2026-06-23T18:49:50Z" -> (known after apply)
!~      id                  = "************************************" -> (known after apply)
+       limits              = (known after apply)
+       main_script_base64  = (known after apply)
!~      modules             = [
-           {
-               content_file   = "/home/runner/work/Charm/Charm/dist/charm/index.js" -> null
-               content_sha256 = "ba9b71550741749400afaa0bbd6f38b92f9607890c4f2b3265e68241823277b9" -> null
-               content_type   = "application/javascript+module" -> null
-               name           = "index.js" -> null
            },
+           {
+               content_file   = "/github/workspace/dist/charm/index.js"
+               content_sha256 = "ba9b71550741749400afaa0bbd6f38b92f9607890c4f2b3265e68241823277b9"
+               content_type   = "application/javascript+module"
+               name           = "index.js"
            },
        ]
!~      number              = 1 -> (known after apply)
!~      source              = "terraform" -> (known after apply)
!~      startup_time_ms     = 0 -> (known after apply)
#        (5 unchanged attributes hidden)
    }

  # cloudflare_workers_custom_domain.site must be replaced
-/+ resource "cloudflare_workers_custom_domain" "site" {
-       environment = "production" -> null # forces replacement
!~      id          = "****************************************" -> (known after apply)
!~      zone_name   = "cloudhub.social" -> (known after apply)
#        (4 unchanged attributes hidden)
    }

  # cloudflare_workers_deployment.site must be replaced
-/+ resource "cloudflare_workers_deployment" "site" {
!~      annotations  = { # forces replacement
!~          workers_message      = "integration" -> "Update workflows"
!~          workers_triggered_by = "deployment" -> (known after apply)
        }
+       author_email = (known after apply)
!~      created_on   = "2026-06-23T18:49:50Z" -> (known after apply)
!~      id           = "************************************" -> (known after apply)
!~      source       = "terraform" -> (known after apply)
!~      versions     = [ # forces replacement
!~          {
!~              version_id = "************************************" -> (known after apply)
#                (1 unchanged attribute hidden)
            },
        ]
#        (3 unchanged attributes hidden)
    }

Plan: 3 to add, 0 to change, 3 to destroy.

📝 Plan generated in Cloudflare Infra #218

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 123d8d2b61

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .github/workflows/sentry-snapshots.yml Outdated
@github-code-quality

github-code-quality Bot commented Jun 23, 2026

Copy link
Copy Markdown

Code Coverage Overview

Languages: JavaScript

JavaScript / code-coverage/vitest

The overall coverage remains at 27%, unchanged from the branch.


Updated June 23, 2026 23:14 UTC
Code Coverage is in Public Preview. Learn more and provide us with your feedback.

@Just-Insane Just-Insane merged commit 677ff43 into integration Jun 23, 2026
17 checks passed
@Just-Insane Just-Insane deleted the workflow-updates branch June 23, 2026 23:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant