Skip to content

Remove dtcc domain from the abused list #39040

Open
only1elii wants to merge 4 commits intoJetBrains:masterfrom
only1elii:master
Open

Remove dtcc domain from the abused list #39040
only1elii wants to merge 4 commits intoJetBrains:masterfrom
only1elii:master

Conversation

@only1elii
Copy link
Copy Markdown

Remove dtcc.edu from the abused list, as this is an Accredited institution, Delaware Technical Community College, with a Computer Science Major.

only1elii and others added 4 commits April 16, 2026 21:31
Adding Delaware Technical Community College DTCC domain informationn
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
This is a real accredited institution with a Computer Science Major
…mputer Science program

Remove 'dtcc.edu' from the list of abused domains as this belongs to Delaware Technical Community College which is an accredited institution with a Computer Science major in the State of Delaware.
Copilot AI review requested due to automatic review settings April 17, 2026 04:41
Copy link
Copy Markdown

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 aims to remove dtcc.edu (Delaware Technical Community College) from the abused domain list so it is no longer treated as an untrusted academic email domain.

Changes:

  • Removed dtcc.edu from lib/domains/abused.txt.
  • Added a new domain entry file for DTCC (but currently in an incorrect location/format).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
lib/domains/dtcc Adds a DTCC entry, but the placement/format will be interpreted incorrectly by the domain compiler/lookup logic.
lib/domains/abused.txt Removes dtcc.edu from the abused list.

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

Comment thread lib/domains/dtcc
@@ -0,0 +1,2 @@
Delaware Technical Community College
dtcc.edu
Copy link

Copilot AI Apr 17, 2026

Choose a reason for hiding this comment

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

lib/domains/dtcc is being added at the root and without a .txt extension. In this repo, domain entries are discovered by path segments under lib/domains (e.g. lib/domains/edu/<domain>.txt for <domain>.edu). Adding a root-level file named dtcc will cause the compiler (src/main/kotlin/swot/Compiler.kt) to treat dtcc as an academic TLD (i.e. .dtcc), which is incorrect and will broaden matches unexpectedly. Move this entry to lib/domains/edu/dtcc.txt and keep the file contents as school name(s) only (the domain should be implied by the path/filename).

Suggested change
dtcc.edu

Copilot uses AI. Check for mistakes.
@philipto
Copy link
Copy Markdown
Collaborator

@only1elii lib/domains/abused.txt: this request has file path that is not expected in a pull request. This repository rules explain what kind of contributions are expected in pull requests. We'll review your request as soon as we can. If the issue concerns a domain that we do not accept due to abuse, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).

@philipto philipto added the problem There is a problem requiring attention label Apr 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

problem There is a problem requiring attention

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants