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

Improve package building and packing for @fluid-experiemental/property-inspector-table #11569

Merged

Conversation

nedalhy
Copy link
Contributor

@nedalhy nedalhy commented Aug 17, 2022

  • Reduce the usage of webpack
  • Using similar webpack configuration to other react libraries in the repo
  • Remove hacks to load the SVGs

Note - for now webpack is used only for running the storybook and bundling SVGs due to the way its consumed in the library itself - this may change in the future, however, it will require further investigation.

@CraigMacomber FYI

@nedalhy
Copy link
Contributor Author

nedalhy commented Aug 17, 2022

@evaliyev PTAL

@github-actions github-actions bot added area: dds: propertydds dependencies Pull requests that update a dependency file labels Aug 17, 2022
@github-actions github-actions bot added the base: main PRs targeted against main branch label Aug 17, 2022
@nedalhy nedalhy marked this pull request as ready for review August 17, 2022 17:27
@nedalhy nedalhy requested review from msfluid-bot and a team as code owners August 17, 2022 17:27
package-lock.json Outdated Show resolved Hide resolved
@nedalhy nedalhy force-pushed the inspector-table/improve_compiling_config branch from c46ddb5 to 2396284 Compare August 22, 2022 14:07
@github-actions github-actions bot removed the dependencies Pull requests that update a dependency file label Aug 22, 2022
@nedalhy
Copy link
Contributor Author

nedalhy commented Aug 24, 2022

@CraigMacomber @DLehenbaue can we merge this PR ?

@CraigMacomber CraigMacomber self-assigned this Aug 25, 2022
…onfig.json

Co-authored-by: Craig Macomber (Microsoft) <42876482+CraigMacomber@users.noreply.github.com>
@CraigMacomber CraigMacomber merged commit eebd39d into microsoft:main Aug 26, 2022
@github-actions
Copy link
Contributor

This commit is queued for merging with the next branch! Please ignore this PR for now. Contact @microsoft/fluid-cr-infra for help.

tyler-cai-microsoft pushed a commit to tyler-cai-microsoft/FluidFramework that referenced this pull request Aug 26, 2022
WayneFerrao pushed a commit to WayneFerrao/FluidFramework that referenced this pull request Aug 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants