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

Update repository URLs #15

Merged
merged 3 commits into from
Aug 7, 2023
Merged

Update repository URLs #15

merged 3 commits into from
Aug 7, 2023

Conversation

steff456
Copy link
Contributor

@steff456 steff456 commented Aug 2, 2023

This PR is part of #13

This PR,

  • Changes url that where pointing to Quansight org and replace it with conda-incubator org.
  • Adds a section for the Code of Conduct in the README.
  • Adds a section for the license in the README.

Copy link
Member

@pavithraes pavithraes left a comment

Choose a reason for hiding this comment

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

Thanks, @steff456! Looks good. :)

README.md Outdated Show resolved Hide resolved
@pavithraes
Copy link
Member

pavithraes commented Aug 7, 2023

@steff456 I'm not sure why the linter is failing :/ If you have a local setup going, could you please run the linter locally to verify?

It's only flagging the README, so I'm also fine with merging with a new issue to remind us to verify the linting setup. :)

Co-authored-by: Pavithra Eswaramoorthy <[email protected]>
@steff456
Copy link
Contributor Author

steff456 commented Aug 7, 2023

Let's wait for the CI to re-run, if it fails I'll check it locally

@anirrudh
Copy link
Contributor

anirrudh commented Aug 7, 2023

kk, this has been linted locally and I've update the PR!

Copy link
Contributor

@anirrudh anirrudh left a comment

Choose a reason for hiding this comment

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

LGTM

@anirrudh anirrudh merged commit 3e0a3ce into conda-incubator:main Aug 7, 2023
@steff456
Copy link
Contributor Author

steff456 commented Aug 7, 2023

Thanks!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done 💪🏾
Development

Successfully merging this pull request may close these issues.

4 participants