Skip to content

fix: included experts directory in build for default expert loading#112

Merged
anand-presidio merged 1 commit intopresidio-oss:mainfrom
magesh-presidio:bugfix/default-experts
Apr 24, 2025
Merged

fix: included experts directory in build for default expert loading#112
anand-presidio merged 1 commit intopresidio-oss:mainfrom
magesh-presidio:bugfix/default-experts

Conversation

@magesh-presidio
Copy link
Copy Markdown
Collaborator

@magesh-presidio magesh-presidio commented Apr 24, 2025

Description

Included the experts folder in .vscodeignore so that it will be preserved during the build process and used to load the default experts in the application.

Type of Change

  • 🐛 Bug fix (non-breaking change which fixes an issue)
  • ✨ New feature (non-breaking change which adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • 📚 Documentation update

Pre-flight Checklist

  • Changes are limited to a single feature, bugfix or chore (split larger changes into separate PRs)
  • Tests are passing (npm test) and code is formatted and linted (npm run format && npm run lint)
  • I have created a changeset using npm run changeset (required for user-facing changes)
  • I have reviewed contributor guidelines

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Apr 24, 2025

⚠️ No Changeset found

Latest commit: ee25598

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@anand-presidio anand-presidio merged commit a1aca7f into presidio-oss:main Apr 24, 2025
2 checks passed
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