Skip to content

chore(ci): restore nucleus downstream testing#5497

Merged
wjhsf merged 4 commits intomasterfrom
wjh/restore-nucleus
Sep 17, 2025
Merged

chore(ci): restore nucleus downstream testing#5497
wjhsf merged 4 commits intomasterfrom
wjh/restore-nucleus

Conversation

@wjhsf
Copy link
Contributor

@wjhsf wjhsf commented Sep 15, 2025

Details

The .nucleus.yaml file was removed in #5464 after we moved publishing fron Nucleus to GitHub Actions. We still need Nucleus for downstream testing, though, so this PR restores the file so we hopefully have the correct configuration. (I'm not 100% sure this is right, the docs aren't super clear.)

If you care about the diff, the first commit in this PR restores the file, then changes are made in subsequent commits.

Does this pull request introduce a breaking change?

  • 😮‍💨 No, it does not introduce a breaking change.
  • 💔 Yes, it does introduce a breaking change.

Does this pull request introduce an observable change?

  • 🤞 No, it does not introduce an observable change.
  • 🔬 Yes, it does include an observable change.

GUS work item

we don't need it for releases, but still need it for downstream testing
@wjhsf wjhsf marked this pull request as ready for review September 15, 2025 21:44
@wjhsf wjhsf requested a review from a team as a code owner September 15, 2025 21:44
@wjhsf wjhsf enabled auto-merge (squash) September 16, 2025 18:24
Copy link
Contributor

@rax-it rax-it left a comment

Choose a reason for hiding this comment

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

Approving it as stopgap until we have a path forward with SFCI.

@wjhsf
Copy link
Contributor Author

wjhsf commented Sep 17, 2025

/nucleus ignore

@salesforce-nucleus
Copy link
Contributor

❌ Error Parsing Command

Missing required argument: reason

/nucleus ignore

Ignore test failures

Options:
  --help        Show help                                              [boolean]
  --reason, -m  A descriptive reason for ignoring the test results
                                                             [string] [required]

Examples:
  /nucleus ignore --reason "Downstream dependency X has a flapping test"

Missing required argument: reason

@wjhsf
Copy link
Contributor Author

wjhsf commented Sep 17, 2025

/nucleus ignore --reason meh

@wjhsf wjhsf merged commit 9ec20a2 into master Sep 17, 2025
7 checks passed
@wjhsf wjhsf deleted the wjh/restore-nucleus branch September 17, 2025 16:36
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