Skip to content
This repository was archived by the owner on Dec 30, 2025. It is now read-only.

Conversation

@amadaluzia
Copy link
Collaborator

@amadaluzia amadaluzia commented Aug 23, 2025

What does this PR do?

This restructures the tree in a way that's seen fit by #3.

Why does this PR exist?

Mostly to just get the thoughts of others in to the restructure before I push to main.

Summary by Sourcery

Enhancements:

  • Add port mapping '6697:6697' to the unrealircd service in docker-compose.yml

@sourcery-ai
Copy link

sourcery-ai bot commented Aug 23, 2025

Reviewer's guide (collapsed on small PRs)

Reviewer's Guide

This PR updates the docker-compose configuration to expose the TLS port (6697) for the unrealircd service, enabling external access in the development environment.

File-Level Changes

Change Details Files
Expose unrealircd TLS port in docker-compose
  • Added ports section under unrealircd service
  • Mapped host port 6697 to container port 6697
docker-compose.yml

Tips and commands

Interacting with Sourcery

  • Trigger a new review: Comment @sourcery-ai review on the pull request.
  • Continue discussions: Reply directly to Sourcery's review comments.
  • Generate a GitHub issue from a review comment: Ask Sourcery to create an
    issue from a review comment by replying to it. You can also reply to a
    review comment with @sourcery-ai issue to create an issue from it.
  • Generate a pull request title: Write @sourcery-ai anywhere in the pull
    request title to generate a title at any time. You can also comment
    @sourcery-ai title on the pull request to (re-)generate the title at any time.
  • Generate a pull request summary: Write @sourcery-ai summary anywhere in
    the pull request body to generate a PR summary at any time exactly where you
    want it. You can also comment @sourcery-ai summary on the pull request to
    (re-)generate the summary at any time.
  • Generate reviewer's guide: Comment @sourcery-ai guide on the pull
    request to (re-)generate the reviewer's guide at any time.
  • Resolve all Sourcery comments: Comment @sourcery-ai resolve on the
    pull request to resolve all Sourcery comments. Useful if you've already
    addressed all the comments and don't want to see them anymore.
  • Dismiss all Sourcery reviews: Comment @sourcery-ai dismiss on the pull
    request to dismiss all existing Sourcery reviews. Especially useful if you
    want to start fresh with a new review - don't forget to comment
    @sourcery-ai review to trigger a new review!

Customizing Your Experience

Access your dashboard to:

  • Enable or disable review features such as the Sourcery-generated pull request
    summary, the reviewer's guide, and others.
  • Change the review language.
  • Add, remove or edit custom review instructions.
  • Adjust other review settings.

Getting Help

@github-advanced-security
Copy link

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

Copy link

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

Hey there - I've reviewed your changes and they look great!


Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@amadaluzia amadaluzia self-assigned this Aug 23, 2025
@amadaluzia amadaluzia added enhancement New feature or request help wanted Extra attention is needed labels Aug 23, 2025
@amadaluzia amadaluzia merged commit a45b2c2 into main Aug 23, 2025
4 of 5 checks passed
@amadaluzia amadaluzia deleted the restructure branch August 23, 2025 01:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

enhancement New feature or request help wanted Extra attention is needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants