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

Changes to margin of content in data science course #1106

Closed
wants to merge 1 commit into from

Conversation

k25kar
Copy link

@k25kar k25kar commented Oct 30, 2024

Issues Identification

Closes: #(issue number)

Description

Summary

A brief description of what this pull request does and why it is needed.

Details

Include any detailed information about the changes in this pull request.

Types of Changes

Please check the boxes that apply

  • Bugfix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update (Documentation content changed)
  • Other (please describe):

Checklist

Please check the boxes that apply

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • My changes do not break the current system and pass all existing test cases
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

Screenshots

If applicable, please attach screenshots of the changes made to the user interface.

Additional Information

Please provide any other information that is relevant to this pull request.

Copy link

netlify bot commented Oct 30, 2024

👷 Deploy Preview for ml-fusion-lab processing.

Name Link
🔨 Latest commit 0c3d51c
🔍 Latest deploy log https://app.netlify.com/sites/ml-fusion-lab/deploys/672240921e250b0008a1c6c1

Copy link

netlify bot commented Oct 30, 2024

Deploy Preview for ml-fusion-lab ready!

Name Link
🔨 Latest commit 0c3d51c
🔍 Latest deploy log https://app.netlify.com/sites/ml-fusion-lab/deploys/672240921e250b0008a1c6c1
😎 Deploy Preview https://deploy-preview-1106--ml-fusion-lab.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

@github-actions github-actions bot 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, k25kar, for creating this pull request and contributing to ML Fusion Lab! 💗

The maintainers will review this Pull Request and provide feedback as soon as possible! 😇
We appreciate your patience and contribution, Keep up the great work! 😀

@vivekvardhan2810
Copy link
Member

i didn't assigned you the issue, and you have created direct Pull Request.

And you haven't created the issue also.

This is not a Good Open Source Practice.

So I am Closing the Pull Request.

Until I assign the issue no one should create or commit the pull request.

please understand.

Please follow the gssoc guidelines properly

This is an open source and Everyone deserves equal opportunity.

And one more thing this project and me are inactive for sometime due to some of my works.

please do work on other projects.

i hope you understand.

@k25kar
Copy link
Author

k25kar commented Oct 30, 2024

oh okay
actually this is my first time learning open source contribution and im learning new stuff
i wasnt aware that u had to open an issue and then make the pr
thank you so much for clarifications

@vivekvardhan2810
Copy link
Member

And one more thing this project and me are inactive for sometime due to some of my works.

please do work on other projects.

i hope you understand.

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