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

Devtool extension for Fluent UI #1162

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

AbhijithKonnayil
Copy link

@AbhijithKonnayil AbhijithKonnayil commented Jan 4, 2025

Add support for Fluent Icons in Dev tools.
#1161

  • I have updated CHANGELOG.md with my changes
  • I have run "dart format ." on the project
  • I have added/updated relevant documentation

Preview

Screenshot 2025-01-05 at 12 16 31 AM

@bdlukaa
Copy link
Owner

bdlukaa commented Jan 5, 2025

I'm not sure you're supposed to commit the build/ folder.

@AbhijithKonnayil
Copy link
Author

AbhijithKonnayil commented Jan 6, 2025

@bdlukaa
The content that you check into your git repository is completely up to you. If you dont need the content of build of devtool extension to be in the code, then make sure that the extenstion web app is always built and included in extension/devtools/build/ when you publish the package
As per docs,

What if I don't want the extension/devtools/build/ contents checked into source control?
As a package author, the content that you check into your git repository is completely up to you. If you want the contents of extension/devtools/build/ to be git ignored, then you'll just need to ensure that the extension web app is always built and included in extension/devtools/build/ when you publish your package. To do so, add the following to a .pubignore file in the extension/devtools/ directory:

!build
This will ensure that, even if the extension/devtools/build directory has been been git ignored, the directory will still be included when publishing the package on pub

https://pub.dev/packages/devtools_extensions#what-if-i-dont-want-the-extensiondevtoolsbuild-contents-checked-into-source-control

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.

2 participants