-
Notifications
You must be signed in to change notification settings - Fork 13.3k
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
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this 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
3409883
to
455eedf
Compare
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. |
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 There are some (correct) matches under: https://github.com/search?q=dlthub&type=repositories but
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? |
That's fair, thanks for the explanation. I'll reopen and merge. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks!
@ahpook does this need to be merged? |
Please confirm this pull request meets the following requirements:
Which change are you proposing?
Curating a new topic or collection
https://github.com/topics/[NAME]
orhttps://github.com/collections/[NAME]
)*.png
image (if applicable) andindex.md
index.md
conform to the Style Guide and API docs: https://github.com/github/explore/tree/main/docsPlease note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.