Skip to content

chore: migrate package manager from npm to pnpm#177

Merged
7w1 merged 6 commits intodevfrom
chore/migrate-to-pnpm
Mar 11, 2026
Merged

chore: migrate package manager from npm to pnpm#177
7w1 merged 6 commits intodevfrom
chore/migrate-to-pnpm

Conversation

@hazre
Copy link
Collaborator

@hazre hazre commented Mar 11, 2026

Description

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

@hazre hazre requested a review from a team March 11, 2026 14:19
@hazre hazre added the internal label Mar 11, 2026
@dozro
Copy link
Contributor

dozro commented Mar 11, 2026

merging this will probably be a problem for #176 bc #176 introduces a new dep

@github-actions
Copy link
Contributor

github-actions bot commented Mar 11, 2026

OpenTofu plan for production

Plan: 2 to add, 0 to change, 2 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 = "571894a54b399bef7a5c40ab051dc6886b2bc8e182571076198b96e31efe3158" -> "cd99ad0c1e17657938f2298c251c151009a0576648daf3f1dbe57a41d49b92bf" # forces replacement
!~          directory             = "/home/runner/work/Sable/Sable/dist" -> "/github/workspace/dist"
#            (1 unchanged attribute hidden)
        }
+       compatibility_flags = (known after apply)
!~      created_on          = "2026-03-10T06:07:19Z" -> (known after apply)
!~      id                  = "************************************" -> (known after apply)
+       limits              = (known after apply)
+       main_script_base64  = (known after apply)
!~      number              = 2 -> (known after apply)
!~      source              = "terraform" -> (known after apply)
+       startup_time_ms     = (known after apply)
#        (4 unchanged attributes hidden)
    }

  # cloudflare_workers_deployment.site must be replaced
-/+ resource "cloudflare_workers_deployment" "site" {
!~      annotations  = { # forces replacement
!~          workers_message      = <<-EOT
              - Merge pull request #126 from SableClient/prepare-release
              - 
              - prepare release
              + chore: migrate package manager from npm to pnpm
            EOT
!~          workers_triggered_by = "deployment" -> (known after apply)
        }
!~      author_email = "cloudflare@sl.sable.moe" -> (known after apply)
!~      created_on   = "2026-03-10T06:07:20Z" -> (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: 2 to add, 0 to change, 2 to destroy.

Warning: Attribute Deprecated

  with cloudflare_workers_custom_domain.site,
  on main.tf line 41, in resource "cloudflare_workers_custom_domain" "site":
  41:   environment = "production"

This attribute is deprecated.

(and one more similar warning elsewhere)

📝 Plan generated in Cloudflare Infra #7

@hazre
Copy link
Collaborator Author

hazre commented Mar 11, 2026

merging this will probably be a problem for #176 bc #176 introduces a new dep

merging this will probably be a problem for #176 bc #176 introduces a new dep

I left a comment on the PR.

@github-actions
Copy link
Contributor

Deploying with  Cloudflare Workers  Cloudflare Workers

Status Preview URL Commit Alias Updated (UTC)
✅ Deployment successful! https://pr-177-sable.raspy-dream-bb1d.workers.dev e037a40 pr-177 Wed, 11 Mar 2026 18:01:33 GMT

@7w1 7w1 merged commit 2d771b4 into dev Mar 11, 2026
8 checks passed
@7w1 7w1 deleted the chore/migrate-to-pnpm branch March 11, 2026 18:04
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.

3 participants