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

feat(core): add project sizes to project graph metadata #22772

Conversation

Cammisuli
Copy link
Member

@Cammisuli Cammisuli commented Apr 10, 2024

Current Behavior

File sizes are not included in the project graph

Expected Behavior

Files sizes are now stored in the file archive file, and then appear in the proper area in the project graph metadata

Related Issue(s)

Fixes #

@Cammisuli Cammisuli requested review from a team as code owners April 10, 2024 18:54
Copy link

vercel bot commented Apr 10, 2024

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

1 Ignored Deployment
Name Status Preview Updated (UTC)
nx-dev ⬜️ Ignored (Inspect) Visit Preview May 9, 2024 7:59pm

@Cammisuli Cammisuli force-pushed the feature/nxa-40-add-file-sizes-to-the-workspace-context-file-archive branch from e752b31 to 4461651 Compare April 10, 2024 18:56
@Cammisuli Cammisuli added the PR status: do not merge This will block a PR from being merged until this tag is removed. label Apr 10, 2024
@Cammisuli Cammisuli force-pushed the feature/nxa-40-add-file-sizes-to-the-workspace-context-file-archive branch from 4461651 to 3bbc036 Compare April 10, 2024 18:57
@Cammisuli Cammisuli removed the PR status: do not merge This will block a PR from being merged until this tag is removed. label Apr 10, 2024
@Cammisuli Cammisuli force-pushed the feature/nxa-40-add-file-sizes-to-the-workspace-context-file-archive branch from 3bbc036 to a6a594c Compare May 1, 2024 16:52
@Cammisuli Cammisuli changed the title feat(core): add file sizes to the files archive feat(core): add project sizes to project graph metadata May 1, 2024
Copy link
Member

Choose a reason for hiding this comment

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

I don't love that this changes public API, but reckon its fine

@Cammisuli Cammisuli force-pushed the feature/nxa-40-add-file-sizes-to-the-workspace-context-file-archive branch from 6f00583 to c2d77c1 Compare May 9, 2024 19:57
@Cammisuli
Copy link
Member Author

We're going with a different direction, and wont be needing this atm.

@Cammisuli Cammisuli closed this May 15, 2024
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.

None yet

2 participants