Skip to content

chore: avoid using multiple out/ directories#2373

Closed
ssbarnea wants to merge 1 commit intomainfrom
chore/buildchain
Closed

chore: avoid using multiple out/ directories#2373
ssbarnea wants to merge 1 commit intomainfrom
chore/buildchain

Conversation

@ssbarnea
Copy link
Member

@ssbarnea ssbarnea commented Dec 9, 2025

No description provided.

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 consolidates the TypeScript compilation output directories from multiple local out/ directories to a centralized out/ directory at the repository root. The ansible-language-server build output is now placed in out/als/ instead of packages/ansible-language-server/out/server/, and the packaged tarball artifact is now generated in out/ship/.

  • Changed TypeScript output directory from ./out/server to ../../out/als
  • Updated all path references across build configurations, debug configurations, CI workflows, and package definitions
  • Removed local out/ directory files and updated exclusion patterns

Reviewed changes

Copilot reviewed 10 out of 14 changed files in this pull request and generated no comments.

Show a summary per file
File Description
packages/ansible-language-server/tsconfig.json Updated outDir to ../../out/als and removed "out" from exclude list
packages/ansible-language-server/package.json Updated file paths, main entry point, and types declaration to reference out/als
packages/ansible-language-server/out/test-als/package.json Removed obsolete test package file from old output directory
packages/ansible-language-server/out/.gitignore Removed obsolete gitignore file from old output directory
packages/ansible-language-server/bin/ansible-language-server Updated require path to reference out/als
packages/ansible-language-server/Taskfile.yml Updated build sources and package output to use centralized out/ship/ directory
packages/ansible-language-server/.vscodeignore Updated VSCode packaging exclusion pattern for out/als
packages/ansible-language-server/.vscode/launch.json Updated debugger configuration name and outFiles path
Taskfile.yml Updated generates path for out/als
.vscodeignore Updated VSCode packaging exclusion pattern at root level
.vscode/launch.json Updated debugger configuration name and outFiles path at root level
.gitignore Added entries for content creator template resources (unrelated to main PR purpose)
.github/workflows/ci.yaml Updated artifact upload path to out/ship/
.c8rc.json Updated code coverage exclusion path to out/als

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants