Addons! #9
No reviewers
Labels
No labels
accessibility
bug
documentation
duplicate
enhancement
good first issue
help wanted
invalid
question
wontfix
No milestone
No project
No assignees
3 participants
Notifications
Due date
No due date set.
Dependencies
No dependencies set
Reference
codingisfun2831t/Crackle!9
Loading…
Reference in a new issue
No description provided.
Delete branch "addons"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Still need a better mod manager and user mod loader, but this PR already includes the
loadModfuncion.Everything automatically gets saved; no ""autoloading""!
Why are there em dashes 🤨
@playforge-coding wrote in #9 (comment):
where? I will admit though, I have copied some parts of Sparkle's example.js. Is that it?
Also, no, I have not used any Generative AI in this PR.
(The screenshot isn't proof. It's just a detail showcasing 0 em dashes)
@d016 wrote in #9 (comment):
Ok, it seems like it is a hyphen, but it is used as an em dash. So probably human.
I've finished the main mod loading functions, although there's no mod manager yet.
I think this is ready for merging. @codingisfun2831t what do you think about this?
@d016 wrote in #9 (comment):
@d016, aren't you supposed to leave a license notice whenever you take considerable portions of MIT-licensed code? Normally I don't care because it's an example, but if it's going to be used to compete with Sparkle, I think that we warrant at least a bit of attribution.
(I'm not a lawyer and this is not legal advice.)
@PPPDUD wrote in #9 (comment):
should we? I did contribute to sparkle, but then again...
@d016 wrote in #9 (comment):
It's a file that's been worked on by multiple people, me included, so it's probably a good idea just to be safe.
@d016 wrote in #9 (comment):
View command line instructions
Checkout
From your project repository, check out a new branch and test the changes.Merge
Merge the changes and update on Forgejo.Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.