Skip to content
This repository has been archived by the owner on Oct 15, 2022. It is now read-only.

Its python cheatsheet #1456

Closed
wants to merge 7 commits into from
Closed

Its python cheatsheet #1456

wants to merge 7 commits into from

Conversation

IamRafy
Copy link

@IamRafy IamRafy commented Aug 29, 2015

@MariagraziaAlastra
Copy link
Member

Hey @IamRafy, thanks for your contribution!
Could you please fill in the PR template for CheatSheets in your first comment? Thank you!

@mintsoft
Copy link
Collaborator

@MariagraziaAlastra wow you got to that quickly! Beat me to it

"val": "Clear parent",
"key": "[Alt] [p]"
"val": "",
"key": "if no scriptFile or command ,Python enters interactive mode "
}, {
"val": "Track to",
"key": "[Ctrl] [t]"
Copy link
Contributor

Choose a reason for hiding this comment

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

Might want to revert the changes to this file. Did you intend to change the blender.json cheatsheet too?

Copy link
Author

Choose a reason for hiding this comment

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

Okay on it. Work in progress.

Regards :)
Mohammed Rafy
On Aug 29, 2015 4:33 PM, "Brian Stoner" notifications@github.com wrote:

In share/goodie/cheat_sheets/json/blender.json
#1456 (comment)
:

     }, {
  •        "val": "Clear parent",
    
  •        "key": "[Alt] [p]"
    
  •        "val": "",
    
  •        "key": "if no scriptFile or command ,Python enters interactive mode "
     }, {
         "val": "Track to",
         "key": "[Ctrl] [t]"
    

Might want to revert the changes to this file. Did you intend to change
the blender.json cheatsheet too?


Reply to this email directly or view it on GitHub
https://github.com/duckduckgo/zeroclickinfo-goodies/pull/1456/files#r38261160
.

Copy link
Contributor

Choose a reason for hiding this comment

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

Awesome 👍

X
^X
Exit
git commit -m "Its Phython Cheat sheet"
@moollaza
Copy link
Member

moollaza commented Sep 8, 2015

@IamRafy sorry for the delay! Please leave us a comment after you make updates in the future so we can see that you've made a change.

It looks like you've accidentally combined the Blender Cheat Sheet with your Python Cheat Sheet? You have sections like "Navigation", "Animation" etc. Can you please remove everything that doesn't belong and leave a comment once the changes are made? Apologies for the delays and difficulties!

@IamRafy
Copy link
Author

IamRafy commented Sep 9, 2015

I have changed the blender Json file a long ago. Anyhow, I haven't
commented in idea page of python sheet related to the changes I make. Is
that correct?
On Sep 9, 2015 2:27 AM, "Zaahir Moolla" notifications@github.com wrote:

@IamRafy https://github.com/IamRafy sorry for the delay! Please leave
us a comment after you make updates in the future so we can see that you've
made a change.

It looks like you've accidentally combined the Blender Cheat Sheet with
your Python Cheat Sheet? You have sections like "Navigation", "Animation"
etc. Can you please remove everything that doesn't belong and leave a
comment once the changes are made? Apologies for the delays and
difficulties!


Reply to this email directly or view it on GitHub
#1456 (comment)
.

@moollaza
Copy link
Member

I have changed the blender Json file a long ago.

@IamRafy it looks like you've somehow combined these changes into your Python Cheat Sheet. Please take a look at it and confirm that everything is correct.

Anyhow, I haven't commented in idea page of python sheet related to the changes I make. Is that correct?

Sorry, I'm not quite sure what you mean by this. You don't need to comment on the Idea Page, if that's what you're asking.

I just need you to go over you cheat sheet here: https://github.com/duckduckgo/zeroclickinfo-goodies/pull/1456/files and make sure that it's all correct. Right now I'm seeing a bunch of content from the Blender Cheat Sheet, which obviously isn't related to Python Syntax.

@moollaza
Copy link
Member

@IamRafy Ping! I'm just checking in to see if you're still interested in finishing this one up? We'd love to see this go live!

There's absolutely no rush, just wanted to make sure you're not stuck on anything.

@IamRafy
Copy link
Author

IamRafy commented Sep 22, 2015

Yes. It's in progress. It's a long one. I will complete this one.
On Sep 22, 2015 6:47 PM, "Zaahir Moolla" notifications@github.com wrote:

@IamRafy https://github.com/IamRafy Ping! I'm just checking in to see
if you're still interested in finishing this one up? We'd love to see this
go live!

There's absolutely no rush, just wanted to make sure you're not stuck on
anything.


Reply to this email directly or view it on GitHub
#1456 (comment)
.

@daxtheduck daxtheduck deployed to beta.duckduckgo.com September 28, 2015 13:42 Active
@daxtheduck daxtheduck deployed to beta.duckduckgo.com September 28, 2015 13:46 Active
@daxtheduck daxtheduck deployed to beta.duckduckgo.com September 28, 2015 14:16 Active
@moollaza
Copy link
Member

@IamRafy I think it would be best to cover just the basics and essentials for now. Take a look at the other Language Cheat Sheets we have (e.g. Lua) which are more concise. The cheat sheet shouldn't really be lengthy and exhaustive, otherwise it will become too much to look at, and hard to navigate.

@IamRafy
Copy link
Author

IamRafy commented Sep 29, 2015

Got it .
On Sep 29, 2015 11:28 PM, "Zaahir Moolla" notifications@github.com wrote:

@IamRafy https://github.com/IamRafy I think it would be best to cover
just the basics and essentials for now. Take a look at the other Language
Cheat Sheets we have (e.g. Lua) which are more concise. The cheat sheet
shouldn't really be lengthy and exhaustive, otherwise it will become too
much to look at, and hard to navigate.


Reply to this email directly or view it on GitHub
#1456 (comment)
.

@IamRafy
Copy link
Author

IamRafy commented Oct 1, 2015

Its done.. The cheat sheet . Check out my PR .

On Tue, Sep 29, 2015 at 11:28 PM, Zaahir Moolla notifications@github.com
wrote:

@IamRafy https://github.com/IamRafy I think it would be best to cover
just the basics and essentials for now. Take a look at the other Language
Cheat Sheets we have (e.g. Lua) which are more concise. The cheat sheet
shouldn't really be lengthy and exhaustive, otherwise it will become too
much to look at, and hard to navigate.


Reply to this email directly or view it on GitHub
#1456 (comment)
.

Best Regards ,
Twitter https://twitter.com/RafyAsarmatta | Facebook
https://www.facebook.com/asarmatta

  • ideabloom.wordpress.com *
    +91-7204945348

@zekiel zekiel closed this Oct 1, 2015
@IamRafy
Copy link
Author

IamRafy commented Oct 1, 2015

Is it all correct Zac ?

On Thu, Oct 1, 2015 at 10:48 PM, Zac Pappis notifications@github.com
wrote:

Closed #1456
#1456.


Reply to this email directly or view it on GitHub
#1456 (comment)
.

Best Regards ,
Twitter https://twitter.com/RafyAsarmatta | Facebook
https://www.facebook.com/asarmatta

  • ideabloom.wordpress.com *
    +91-7204945348

@moollaza
Copy link
Member

moollaza commented Oct 1, 2015

@IamRafy We closed this because it looks like you continued it in #1622 ?

@moollaza
Copy link
Member

moollaza commented Oct 1, 2015

Both of these look the same though, but this PR has broken JSON syntax...

@IamRafy
Copy link
Author

IamRafy commented Oct 2, 2015

So simply I wasted my time .. ! It was a new file . it was not a
blender.Json.
On Oct 2, 2015 12:05 AM, "Zaahir Moolla" notifications@github.com wrote:

Both of these look the same though, but this PR has broken JSON syntax...


Reply to this email directly or view it on GitHub
#1456 (comment)
.

@moollaza
Copy link
Member

moollaza commented Oct 2, 2015

@IamRafy sorry about the confusion! You certainly haven't wasted your time. I've responded to you on Slack, explaining what we thought was going on. We can definitely move forward with either PR, it seems like we don't need both though?

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

Successfully merging this pull request may close these issues.

None yet

9 participants