Skip to content

🔨 fixed fetch contribution script #1237

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

Merged
merged 1 commit into from
Oct 5, 2021

Conversation

Ankit1598
Copy link
Contributor

Description

Fixed the organization link as previously the code was using a old layout page link

Fixes #1234

Type of change

  • Bug fix (non-breaking change which fixes an issue)

Checklist:

  • My code follows the style guidelines(Clean Code) of this project
  • I have performed a self-review of my own code

@ghost
Copy link

ghost commented Sep 30, 2021

Changes to your CodeSee Architecture Map:

View a CodeSee Map of these changes

Legend

CodeSee Map Legend

Copy link
Owner

@HarshCasper HarshCasper left a comment

Choose a reason for hiding this comment

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

This looks quite good to me. I would be happy to merge this in the current state 😄

I have one request: Is this possible to convert this into a package? It would be easy to set up through a single click and we can convert this into a CLI package for our purpose.

For future purpose, I will distribute it through the PyPi packaging for easy install and usage.

@HarshCasper HarshCasper merged commit d6613cf into HarshCasper:master Oct 5, 2021
@github-actions
Copy link

github-actions bot commented Oct 5, 2021

@Ankit1598 Hurray, your PR got merged 🥳. Thanks a bunch. Feel free to take another Issue.💖

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: Error in Fetch Contribution script
2 participants