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

Add inferred target #1153

Closed
wants to merge 0 commits into from
Closed

Add inferred target #1153

wants to merge 0 commits into from

Conversation

jbadeau
Copy link

@jbadeau jbadeau commented Oct 17, 2024

This PR provides support for

  • init generator to pre-configure plugin
  • inferred container target to minimise required configuration

@mpsanchis
Copy link

🔥

@jbadeau
Copy link
Author

jbadeau commented Oct 17, 2024

There is an issue with nx v20 as they have removed api i.e. createNodes

not sure what to do about this yet.

@jbadeau
Copy link
Author

jbadeau commented Oct 30, 2024

@gperdomor ping

Copy link

nx-cloud bot commented Nov 4, 2024

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 7c250aa. 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


🟥 Failed Commands
nx affected -t lint test build
✅ Successfully ran 1 target

Sent with 💌 from NxCloud.

Copy link
Owner

@gperdomor gperdomor left a comment

Choose a reason for hiding this comment

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

Hi @jbadeau... Thanks for this PR... createNodes it's deprecated but is not removed yet, so we're good for now :D... Please rebase this and fix lint issues 🙏🏻

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.

3 participants