Skip to content

Conversation

@pepone
Copy link
Member

@pepone pepone commented Sep 29, 2025

No description provided.

Copy link

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 Python demo for the Ice framework that demonstrates how to define and use custom exceptions in Slice. The demo shows how to define a custom exception type, throw it from a server implementation, and catch it in a client application.

Key changes:

  • Added complete custom error demo with client/server architecture
  • Defined GreeterException with error codes in Slice interface definition
  • Implemented exception handling patterns for both custom and system exceptions

Reviewed Changes

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

Show a summary per file
File Description
python/README.md Added table entry for the new Ice Custom Error demo
python/Ice/customError/slice/Greeter.ice Slice interface definition with custom GreeterException and error enum
python/Ice/customError/server/requirements.txt Server dependencies configuration for zeroc-ice nightly builds
python/Ice/customError/server/main.py Server application entry point with Ice communicator setup
python/Ice/customError/server/chatbot.py Chatbot servant implementation demonstrating custom exception throwing
python/Ice/customError/client/requirements.txt Client dependencies configuration for zeroc-ice nightly builds
python/Ice/customError/client/main.py Client application demonstrating custom exception handling
python/Ice/customError/README.md Complete setup and usage documentation for the demo
python/Ice/customError/.gitignore Ignores generated Slice compilation artifacts

Copy link
Member

Choose a reason for hiding this comment

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

Should not commit this file

@pepone pepone merged commit 0f2baef into zeroc-ice:main Sep 30, 2025
13 of 15 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