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

Upgrades to Scratch 2.0 (and ScratchX) #41

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

Conversation

raulperula
Copy link
Contributor

Changes:

  • Upgraded the files format to follow the standard PEP8 Python format.
  • Upgraded the gitignore file with more auto-generated files (e.g. Eclipse IDE).
  • Upgraded the auto-generation of the "command_src" file based on the source code of Scratch published in GitHub.
  • Upgraded the Kurt pkg and the Scratch20 plugin to load extensions (own designed or by other providers such as Lego Wedo).

@tjvr
Copy link
Owner

tjvr commented Aug 1, 2017

Thanks for working on this! :D

I'm busy this week, but I'll have a look through your changes next week. :-)

@tjvr
Copy link
Owner

tjvr commented Aug 8, 2017

I’ll get to reviewing this in a bit, but before I do, please could you drop the PEP8 changes (eg by rebasing to remove those commits)? They make the diff harder to read, and I’m not particularly interested in changing the code style of the project. Sorry! :-)

@CosmicWebServices
Copy link

@raulperula
Copy link
Contributor Author

raulperula commented Aug 11, 2017

Sorry, my time is very limited in this moment due to project results. Format changes are very slight, I am sure you are able to see the real changes by reading the commits. I do not remember if you could select only those commits that you want to incorporate.

However, thanks for the information @CosmicWebServices and I could try to do it if I have some time. Sorry again!

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.

3 participants