Skip to content

🎨 Enabled nx cloud#28512

Merged
9larsons merged 1 commit into
mainfrom
reenable-nx-cloud
Jun 11, 2026
Merged

🎨 Enabled nx cloud#28512
9larsons merged 1 commit into
mainfrom
reenable-nx-cloud

Conversation

@9larsons

Copy link
Copy Markdown
Contributor

no ref

We're now on the OSS plan w/ nx cloud! Also re-enabling analytics.

no ref

We're now on the OSS plan w/ nx cloud! Also re-enabling analytics.
@9larsons 9larsons enabled auto-merge (squash) June 11, 2026 17:00
@coderabbitai

coderabbitai Bot commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 27610fa5-c77a-4e7d-90bb-26e089baaa7d

📥 Commits

Reviewing files that changed from the base of the PR and between dbfe5a0 and cc262f4.

📒 Files selected for processing (1)
  • nx.json

Walkthrough

This PR updates the Nx Cloud configuration in nx.json by enabling analytics telemetry and adding an Nx Cloud project ID. The change modifies lines 58–59 to set the analytics flag from false to true and introduces the nxCloudId configuration property, registering this repository with a specific Nx Cloud workspace.

Possibly related PRs

  • TryGhost/Ghost#28110: Both PRs modify nx.json to configure Nx Cloud by adding/setting nxCloudId and adjusting Nx Cloud-related settings.
  • TryGhost/Ghost#28151: Both PRs modify nx.json Nx Cloud settings—this PR enables analytics and adds nxCloudId, while the other PR removes nxCloudId and disables Nx Cloud.

Suggested labels

ok to merge for me

Suggested reviewers

  • troyciesco
🚥 Pre-merge checks | ✅ 3 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Title check ❓ Inconclusive The title is vague and doesn't clearly describe the main change; it uses an emoji and lacks specificity about what was actually modified. Use a more descriptive title that clearly states the change, such as 'Enable Nx Cloud analytics and configure OSS plan' or 'Configure Nx Cloud with analytics enabled'.
✅ Passed checks (3 passed)
Check name Status Explanation
Description check ✅ Passed The description is related to the changeset, mentioning Nx Cloud and analytics which match the actual configuration updates made.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch reenable-nx-cloud

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@nx-cloud

nx-cloud Bot commented Jun 11, 2026

Copy link
Copy Markdown

View your CI Pipeline Execution ↗ for commit cc262f4

Command Status Duration Result
nx build @tryghost/portal ✅ Succeeded <1s View ↗
nx build @tryghost/sodo-search ✅ Succeeded <1s View ↗
nx build @tryghost/comments-ui ✅ Succeeded <1s View ↗
nx build @tryghost/announcement-bar ✅ Succeeded <1s View ↗
nx build @tryghost/activitypub ✅ Succeeded 1s View ↗
nx build @tryghost/admin-toolbar ✅ Succeeded 1s View ↗
nx build @tryghost/signup-form ✅ Succeeded <1s View ↗
nx run @tryghost/admin-x-settings:test:acceptance ✅ Succeeded 9m 47s View ↗
Additional runs (10) ✅ Succeeded ... View ↗

💡 Verify your cache is correct by running tasks in a sandbox. Read docs ↗


☁️ Nx Cloud last updated this comment at 2026-06-11 17:14:33 UTC

@9larsons 9larsons merged commit 9169165 into main Jun 11, 2026
51 checks passed
@9larsons 9larsons deleted the reenable-nx-cloud branch June 11, 2026 17:22
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.

1 participant