Skip to content

Moved the Slice Compiler Definitions from icerpc-slice -> slicec - #45

Merged
InsertCreativityHere merged 1 commit into
icerpc:mainfrom
InsertCreativityHere:remove-slice-compiler-definitions
Mar 19, 2026
Merged

InsertCreativityHere merged 1 commit into
icerpc:mainfrom
InsertCreativityHere:remove-slice-compiler-definitions

Conversation

@InsertCreativityHere

Copy link
Copy Markdown
Member

Now that the Slice compiler definitions live in slicec (icerpc/slicec#737), we should delete them from icerpc-slice.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Removes the Slice compiler definition files from this repository now that they live in slicec, and updates the top-level README to reflect the remaining directory layout for consumers cloning these Slice definitions.

Changes:

  • Removed the Compiler/* Slice definition files (compiler symbol model and code generator interfaces).
  • Updated README.md to drop the Compiler and top-level Ice rows and point Ice interop definitions to IceRpc/Ice.

Reviewed changes

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

File Description
README.md Updates the directory table to match the new layout (no Compiler/, Ice interop under IceRpc/Ice).
Compiler/SyntaxElements.slice Removes compiler symbol/type model definitions from this repo.
Compiler/DocComment.slice Removes doc comment model definitions from this repo.
Compiler/CodeGenerator.slice Removes code generator interface and related types from this repo.

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

@InsertCreativityHere
InsertCreativityHere merged commit 5eb3c8a into icerpc:main Mar 19, 2026
6 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.

4 participants