Skip to content
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

Delete the old unused logo file #936

Merged
merged 2 commits into from
Jul 10, 2024
Merged

Delete the old unused logo file #936

merged 2 commits into from
Jul 10, 2024

Conversation

asarraf
Copy link
Contributor

@asarraf asarraf commented Jul 10, 2024

dbt-logo.png is the old logo and hence can be deleted.

resolves #890
Thanks @dbeatty10 for raising the issue.

Problem

In the dbt-utils repo we are still preserving the old logo.
We no longer use this logo; the new logo has been replaced everywhere.
This PR works on the decommissioning of the old logo completely from the Repo

Search Results

Screenshot 2024-07-09 at 9 09 48 PM Screenshot 2024-07-09 at 9 09 34 PM

Solution

We are deleting the etc Directory from the Repo -> https://github.com/dbt-labs/dbt-utils

Checklist

  • This code is associated with an issue which has been triaged and accepted for development.
  • I have read the contributing guide and understand what's expected of me
  • I have run this code in development and it appears to resolve the stated issue
  • This PR includes tests, or tests are not required/relevant for this PR
  • I have updated the README.md (if applicable)

asarraf added 2 commits July 9, 2024 21:12
`dbt-logo.png` is the old logo and hence can be deleted.
Copy link
Contributor

@dbeatty10 dbeatty10 left a comment

Choose a reason for hiding this comment

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

Thank you @anks2024 ! 🏆

@dbeatty10 dbeatty10 added this pull request to the merge queue Jul 10, 2024
Merged via the queue into dbt-labs:main with commit 680aaef Jul 10, 2024
8 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.

Remove file of old dbt logo
2 participants