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

docs(misc): Add Manage Library Versions recipe #19666

Merged

Conversation

ndcunningham
Copy link
Contributor

@ndcunningham ndcunningham commented Oct 17, 2023

Preview: https://nx-dev-git-fork-ndcunningham-docs-manage-library-ve-c0e82d-nrwl.vercel.app/recipes/module-federation/manage-library-versions

Current Behavior

No Recipe for Manging library versions with Module Federation

Expected Behavior

We should have a recipe, of how Nx manages library versions

Related Issue(s)

Fixes #

@ndcunningham ndcunningham added the scope: docs Issues related to generic docs label Oct 17, 2023
@ndcunningham ndcunningham self-assigned this Oct 17, 2023
@ndcunningham ndcunningham requested review from a team as code owners October 17, 2023 19:01
@ndcunningham ndcunningham requested a review from bcabanes October 17, 2023 19:01
@vercel
Copy link

vercel bot commented Oct 17, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
nx-dev ✅ Ready (Inspect) Visit Preview Oct 19, 2023 2:56pm

Copy link
Collaborator

@isaacplmann isaacplmann left a comment

Choose a reason for hiding this comment

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

I really like the content, but this is a concept guide, not a recipe. It should be moved up into the (yet to be created) Module Federation section under concepts.

Also, we should make nice images in Figma that match our color scheme and work on both light and dark themes. (The images can be done in a separate PR though).

Copy link
Contributor

@Coly010 Coly010 left a comment

Choose a reason for hiding this comment

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

LGTM! 🎉 This explained it very well. Some minor comments

Copy link
Collaborator

@isaacplmann isaacplmann left a comment

Choose a reason for hiding this comment

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

I agree with Colum's comments. I'm approving so that you can merge as soon as you make those changes.

@ndcunningham ndcunningham force-pushed the docs/manage-library-versions-mfe branch from a4ce926 to e088ef9 Compare October 19, 2023 14:50
@ndcunningham ndcunningham enabled auto-merge (squash) October 19, 2023 14:52
@ndcunningham ndcunningham merged commit fb37e0d into nrwl:master Oct 19, 2023
2 checks passed
@ndcunningham ndcunningham deleted the docs/manage-library-versions-mfe branch October 19, 2023 14:57
@github-actions
Copy link

This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 25, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
scope: docs Issues related to generic docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants