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

chore: remove .DS_Store files and improve documentation #267

Merged
merged 2 commits into from
Jun 24, 2024

Conversation

cdragos
Copy link
Contributor

@cdragos cdragos commented Jun 24, 2024

Description

I've made a couple of housekeeping changes in this PR:

  1. Cleaned up some pesky .DS_Store files that were hanging around
  2. Did some touch-ups on our docs:
  • Fixed a grammar hiccup in the "Library Structure" section
  • Corrected the "discuss.roboflow.com" URL (oops!)
  • Tidied up the HTML for our social media icons at the bottom

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • This change requires a documentation update

Docs

  • Docs updated? What were the changes:

  • Corrected some grammatical issues

  • Fixed typo in the "discuss.roboflow.com" URL

  • Improved HTML structure for social media icons

@cdragos
Copy link
Contributor Author

cdragos commented Jun 24, 2024

@iurisilvio Could you please review this PR when you have a moment? Thanks!

@cdragos cdragos changed the title Chore/remove ds store files chore: remove .DS_Store files and improve documentation Jun 24, 2024
@iurisilvio iurisilvio self-requested a review June 24, 2024 07:22
@iurisilvio iurisilvio self-assigned this Jun 24, 2024
@iurisilvio
Copy link
Contributor

Thanks! 🚢

@cdragos
Copy link
Contributor Author

cdragos commented Jun 24, 2024

@iurisilvio

Thanks! 🚢

@iurisilvio

Thanks for the approve!
I don’t have the rights to merge this PR.

@iurisilvio iurisilvio merged commit dbed76f into roboflow:main Jun 24, 2024
6 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.

2 participants