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

Add support for custom file associations #111

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Dhravya
Copy link

@Dhravya Dhravya commented Jun 2, 2024

Related to #103

Introduces custom file associations for dotenv files through settings.json, enhancing flexibility and user configuration options.

  • Package.json Updates:

    • Adds a new configuration setting dotenv.files.associations to allow users to define custom file associations for dotenv files.
    • Updates activationEvents to include onLanguage:dotenv, ensuring the extension activates for custom file associations.
  • Code Logic Enhancement:

    • Modifies lib/fileAssociations.js to read custom file associations from settings.json and merges them with default associations. This allows for dynamic association of files with the dotenv language based on user preferences.
  • Documentation:

    • Updates README.md to include instructions on how to define custom file associations for dotenv files in settings.json, providing clear guidance for users to leverage this new feature.

I needed this feature for allowing files like .dev.vars to be cloaked during stream. Used copilot workspace to make a PR

For more details, open the Copilot Workspace session.

@remcohaszing
Copy link

Please have a look at #105 first. It solves a deeper issue of which #103 is a symptom.

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