Skip to content

Conversation

@djarbz
Copy link
Contributor

@djarbz djarbz commented Nov 7, 2025

Description

Exposes the metadata of the selected IDEs for use in the main template.
Also adds tests to verify that the output metadata matches the "default" mappings.

Type of Change

  • New module
  • New template
  • Bug fix
  • Feature/enhancement
  • Documentation
  • Other

Module Information

Path: registry/coder/modules/jetbrains
New version: v1.2.0
Breaking change: [ ] Yes [x] No

Testing & Validation

  • [N/A] Tests pass (bun test)
  • Code formatted (bun fmt)
  • Changes tested locally

Related Issues

None

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a new ide_metadata output to the JetBrains module that exposes metadata for selected IDEs, making it easier for users to reference IDE information in their templates. The version is bumped to v1.2.0.

  • Adds ide_metadata output that returns a map of selected IDEs with their name, icon, and build information
  • Includes comprehensive test coverage with validation tests and output tests
  • Updates documentation with usage examples showing how to reference the new output

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
registry/coder/modules/jetbrains/main.tf Adds new ide_metadata output that maps selected IDEs to their metadata from local.options_metadata
registry/coder/modules/jetbrains/jetbrains.tftest.hcl Adds test configuration variables and comprehensive test cases to validate the new output behavior
registry/coder/modules/jetbrains/README.md Updates module version to 1.2.0 and adds documentation section with example usage of the ide_metadata output

@DevelopmentCats
Copy link
Contributor

@djarbz other than the above everything looks good to me.

@DevelopmentCats DevelopmentCats merged commit f304201 into coder:main Nov 14, 2025
4 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.

2 participants