Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Addon Test: Error when addon interactions exists #29434

Conversation

valentinpalkovic
Copy link
Contributor

@valentinpalkovic valentinpalkovic commented Oct 23, 2024

This PR introduces a preset annotation to addon test to check if addon interactions exists, and if it does, it will throw an error.

it also fixes the entry check for addon actions order, which wasn't working before.

@valentinpalkovic valentinpalkovic self-assigned this Oct 23, 2024
@valentinpalkovic valentinpalkovic added ci:daily Run the CI jobs that normally run in the daily job. addon: test feature request labels Oct 23, 2024
Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

3 file(s) reviewed, 2 comment(s)
Edit PR Review Bot Settings | Greptile

code/addons/interactions/src/preset.ts Show resolved Hide resolved
Copy link

nx-cloud bot commented Oct 23, 2024

☁️ Nx Cloud Report

CI is running/has finished running commands for commit c02fe50. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this CI Pipeline Execution


✅ Successfully ran 1 target

Sent with 💌 from NxCloud.

@valentinpalkovic valentinpalkovic changed the title Addon Test: Error when addon interactions exists"" Addon Test: Error when addon interactions exists Oct 24, 2024
Base automatically changed from unified-ui-testing to next October 24, 2024 09:18
@valentinpalkovic valentinpalkovic force-pushed the revert-29433-revert-29338-yann/addon-interactions-conflict branch from 79f711f to f7c632d Compare October 24, 2024 12:30
@valentinpalkovic valentinpalkovic merged commit 6098c01 into next Oct 25, 2024
109 checks passed
@valentinpalkovic valentinpalkovic deleted the revert-29433-revert-29338-yann/addon-interactions-conflict branch October 25, 2024 13:40
@github-actions github-actions bot mentioned this pull request Oct 25, 2024
11 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
addon: test ci:daily Run the CI jobs that normally run in the daily job. feature request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants