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

Update Comment blocks #175

Merged
merged 9 commits into from
Mar 12, 2024
Merged

Update Comment blocks #175

merged 9 commits into from
Mar 12, 2024

Conversation

fath11
Copy link
Contributor

@fath11 fath11 commented Feb 3, 2024

Changed the banner to match the block's colour palate
I moved Comment blocks to a new Turbowarp folder.

Also, @sylarhcn, can you remove my name from the credits section of Comment blocks and Big Int? I hope my name won't be displayed in any Turbowarp ported extensions I committed.

@sylarhcn
Copy link
Contributor

you dont want show your name? why?

@fath11
Copy link
Contributor Author

fath11 commented Feb 27, 2024

you dont want show your name? why?

just ones that i do not make any significant changes to. porting those extensions doesn't take much effort so i rather not have it under my name

@fath11
Copy link
Contributor Author

fath11 commented Feb 27, 2024

also can i know why you guys choose LGPL v2.1?

@sylarhcn
Copy link
Contributor

sylarhcn commented Mar 12, 2024

also can i know why you guys choose LGPL v2.1?

I think there's no particular reason, chose it by default maybe. :D

@sylarhcn sylarhcn merged commit 4c25c32 into Gandi-IDE:main Mar 12, 2024
MoreBugOfDog pushed a commit to MoreBugOfDog/custom-extension that referenced this pull request Mar 24, 2024
* Create CommentBlocks.js

* Removed description blocks and reduce the use of block functions

* allow reporter to be droped everywhere

* comment blocks.js now 100% compatible with its tw version

* renamed a few blocks

* Removed unneeded translations

* Changed extension cover

* Moved commentblocks.js and its asset to a Turbowarp folder

* Update comment blocks.js
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