Skip to content

feat: add data-load-tool topic #4811

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

feat: add data-load-tool topic #4811

wants to merge 1 commit into from

Conversation

joscha
Copy link

@joscha joscha commented Jun 30, 2025

Please confirm this pull request meets the following requirements:

Which change are you proposing?

  • Suggesting edits to an existing topic or collection
  • Curating a new topic or collection
  • Something that does not neatly fit into the binary options above

Curating a new topic or collection

  • I've formatted my changes as a new folder directory, named for the topic or collection as it appears in the URL on GitHub (e.g. https://github.com/topics/[NAME] or https://github.com/collections/[NAME])
  • My folder contains a *.png image (if applicable) and index.md
  • All required fields in my index.md conform to the Style Guide and API docs: https://github.com/github/explore/tree/main/docs

Please replace this line with an explanation of why you think this topic or collection should be curated.


Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

@Copilot Copilot AI review requested due to automatic review settings June 30, 2025 12:30
@joscha joscha requested a review from a team as a code owner June 30, 2025 12:30
Copy link
Contributor

@Copilot 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 curates a new topic collection for a Python library called "DLT" by adding a new folder with metadata and documentation.

  • Adds a new topic collection metadata file for the dlt Python library
  • Includes project details including creation, aliases, description, and URLs
Comments suppressed due to low confidence (1)

topics/data-load-tool/index.md:2

  • [nitpick] Consider verifying the naming convention for 'dltHub' to ensure it aligns with any established branding guidelines (e.g., 'DLTHub').
created_by: dltHub

@joscha joscha changed the title feat: add dlt feat: add data-load-tool topic Jun 30, 2025
@joscha joscha force-pushed the patch-1 branch 3 times, most recently from 3409883 to 455eedf Compare June 30, 2025 12:50
@ahpook
Copy link
Contributor

ahpook commented Jul 14, 2025

It looks like all of the repos in this topic are under the namespace of one commercial company, planet-a-ventures - so I think it falls under the "self-promotion" clause.
https://github.com/topics/data-load-tool

@ahpook ahpook closed this Jul 14, 2025
@joscha
Copy link
Author

joscha commented Jul 15, 2025

It looks like all of the repos in this topic are under the namespace of one commercial company, planet-a-ventures - so I think it falls under the "self-promotion" clause. https://github.com/topics/data-load-tool

Hi @ahpook , thank you for looking into this.

There are loads of repositories which have code for the same tool that are not in our org. Planet A has no affiliation with DLT, so it is definitely not self promotion. The reason I added the tag is because at current time there is no tag binding them all together, so you can't navigate between different repositories that are all spanning the same topic.

There are some under dlt: https://github.com/search?q=dlt&type=repositories but the problem there is that DLT as an acronym is too short to meaningfully connect the right repositories. Lot's of different organizations use the dlt acronym, not at last Delta Live Tables which is at least equally as big as the one from dltHub. Introducing a dlt topic therefore didn't really make sense as it wouldn't meaningfully cover the right space of the topic. Hence why I went with data-load-tool.

There are some (correct) matches under: https://github.com/search?q=dlthub&type=repositories but dltHub is the name of the organization maintaining/producing DLT, hence it's not ideal either.

  1. It makes sense to have a common topic that can be used to label/tag all existing repositories that have something to do with dltHub's DLT
  2. It does not make sense to lump them all together with other topics that also happen to use the acronym DLT

Can you please give some guidance to navigate this issue @ahpook, so I can introduce a topic that people can use in the future to tag correctly?

@ahpook
Copy link
Contributor

ahpook commented Jul 17, 2025

That's fair, thanks for the explanation. I'll reopen and merge.

@ahpook ahpook reopened this Jul 17, 2025
Copy link
Contributor

@ahpook ahpook left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@joscha
Copy link
Author

joscha commented Jul 21, 2025

@ahpook does this need to be merged?

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.

2 participants