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

Initialize pinecone package, clean up deprecated ioutil methods, Replace URL with correct GitHub Org URL #56

Merged
merged 1 commit into from Dec 8, 2023

Conversation

gregandcin
Copy link
Collaborator

I went through and did several things

  1. Initialized the project with go mod to make it easier for other people to contribute to the project without jumping through simple setup hoops
  2. Added Go related .gitignore entries to make sure no one commits a bad file
  3. Replaced ioutil methods with io and os calls since ioutil has been deprecated since Go 1.16
  4. Replaced OfficialTeamUIX with Xbox-Preservation-Project in the URL calls so it doesn't try to call a bad URL in the future

Copy link
Collaborator

@MrMilenko MrMilenko left a comment

Choose a reason for hiding this comment

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

Looks good, thank you!

@MrMilenko MrMilenko merged commit 9091083 into Xbox-Preservation-Project:main Dec 8, 2023
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.

None yet

2 participants