◐ Shell
reader mode source ↗
Skip to content

added a basic swap program#22

Open
SrivastavaSatyam wants to merge 2 commits into
Akuli:masterfrom
SrivastavaSatyam:master
Open

added a basic swap program#22
SrivastavaSatyam wants to merge 2 commits into
Akuli:masterfrom
SrivastavaSatyam:master

Conversation

@SrivastavaSatyam

@SrivastavaSatyam SrivastavaSatyam commented Oct 1, 2020

Copy link
Copy Markdown

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

@Akuli

Akuli commented Oct 2, 2020

Copy link
Copy Markdown
Owner

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

@SrivastavaSatyam SrivastavaSatyam left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Hide comment

check now ,are you fine with it.

@Akuli

Akuli commented Oct 2, 2020

Copy link
Copy Markdown
Owner

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

SrivastavaSatyam commented Oct 3, 2020

Copy link
Copy Markdown
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

@ENVIRYO2112VIT ENVIRYO2112VIT left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Hide comment

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

1 more reviewer

@ENVIRYO2112VIT ENVIRYO2112VIT ENVIRYO2112VIT approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@SrivastavaSatyam @Akuli @ENVIRYO2112VIT @subhanshugithub