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.

Sign up for GitHub

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

Jump to bottom

added a basic swap program #22

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

added a basic swap program #22

wants to merge 2 commits into from

Conversation

SrivastavaSatyam
Copy link

Just, a few line of code....and number swaped .For the one new to the python

@Akuli
Copy link
Owner

Akuli commented Oct 2, 2020

Can you put it to one of the .md files along with an explanation of what it does?

Even better, can you break the code and turn this into a "fix this code" exercise?

@SrivastavaSatyam
Update swap.py
73eae10 
turned the code into an exercise.
now ,is it ok?
SrivastavaSatyam
SrivastavaSatyam commented Oct 2, 2020
View reviewed changes
Copy link
Author

@SrivastavaSatyam SrivastavaSatyam left a comment

Choose a reason for hiding this comment

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

check now ,are you fine with it.

@Akuli
Copy link
Owner

Akuli commented Oct 2, 2020

I guess I can move the code into some .md file myself. I don't know when I'll have the time to do that though.

@SrivastavaSatyam
Copy link
Author

I'll try to add, if I could in the way u had .

ENVIRYO2112VIT
ENVIRYO2112VIT approved these changes Apr 18, 2021
View reviewed changes
Copy link

@ENVIRYO2112VIT ENVIRYO2112VIT left a comment

Choose a reason for hiding this comment

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

Keep some comments to give a hint for how to swap too

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ENVIRYO2112VIT ENVIRYO2112VIT ENVIRYO2112VIT approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@SrivastavaSatyam @Akuli @ENVIRYO2112VIT @subhanshugithub