-
-
Notifications
You must be signed in to change notification settings - Fork 377
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
Merge feature branch into master (before release). #7307
Draft
Moderocky
wants to merge
236
commits into
master
Choose a base branch
from
dev/feature
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+42,564
−32,209
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Update lang package to new annotation conventions, new warning conventions, and use try-with-resources for loggers * Apply suggestions from code review Co-authored-by: Patrick Miller <[email protected]> * Requested changes --------- Co-authored-by: Moderocky <[email protected]> Co-authored-by: Patrick Miller <[email protected]>
Require Java 17 Co-authored-by: sovdee <[email protected]>
* Allows for multiple return types in ExpressionEntryData - Changes the constructors to allow for multiple return types - Changes the calls to SkriptParser#parseExpression to pass in the array of types * Changes list to array and redirects existing constructors - Changes returnTypes field to an array - Adds 2 new constructors, one which has the flags parameter and the other that defaults to ALL_FLAGS --------- Co-authored-by: sovdee <[email protected]>
* init commit innit * minor touch ups * oops * fix annotation * oopsie * fixes * removed unused import * update annotation * added text --------- Co-authored-by: sovdee <[email protected]>
* Adds tame syntax * Adds test * Adds Skript doc annotations * NotSoDelayed Suggestions Co-authored-by: NotSoDelayed <[email protected]> * Suggestions - Removed 'de' as an option to untame a tameable entity - Adds untaming to test - Deletes value within variables instead of killing * Fixes test * Changed tests slightly * Auggestion Co-authored-by: NotSoDelayed <[email protected]> * Suggestions * Changes to JetBrains nullable annotation --------- Co-authored-by: NotSoDelayed <[email protected]> Co-authored-by: sovdee <[email protected]>
* Add condition type to fix parsing order * Add JUnit test * Move java imports backs * Move java imports backs * Complete tests * Change tests * Complete tests * Update src/test/skript/tests/regressions/5804-is-burning.sk
* Add optional ascending, descending sort order to SortEff * Add version to Since annotation for explicit sort order * Change toString() to reflect change in syntax * Add missing space in EffSort#toString * Change "explicit order" to "sort order" for clarity * Revert "Change "explicit order" to "sort order" for clarity" This reverts commit ae11d23. * Update src/main/java/ch/njol/skript/effects/EffSort.java --------- Co-authored-by: Moderocky <[email protected]>
* Add duration conversion methods. * Add TemporalAmount & TemporalUnit methods.
* Added on piglin bartering * Added bartering input * Added bartering output * Minor formatting/documentation fixes * Fixed IDE using wildcard import * Epically forgot space * Fixed review comments * Okay, should be good now * Added junit tests * Fix tests for <1.16 versions * uhh * Only register exprs if event exists * Remove plagiarism * Minor spelling mistake * Fixed errors * Fixed no completion * oopsie * tried parsing section * forgot "to console" * move to file loading * rename file * remove broadcast * Update src/main/java/ch/njol/skript/expressions/ExprBarterDrops.java Co-authored-by: Patrick Miller <[email protected]> * Update src/main/java/ch/njol/skript/expressions/ExprBarterDrops.java Co-authored-by: Patrick Miller <[email protected]> * Update src/main/java/ch/njol/skript/expressions/ExprBarterInput.java Co-authored-by: Patrick Miller <[email protected]> * Update src/test/skript/junit/EvtPiglinBarterTest.sk Co-authored-by: Patrick Miller <[email protected]> * Apply suggestions from code review Didn't realise this existed until now. Oops! Co-authored-by: Patrick Miller <[email protected]> * Fixed review stuff * Extra nls Co-authored-by: Patrick Miller <[email protected]> * oops * fixes * moar fixes * thanks pickle --------- Co-authored-by: Moderocky <[email protected]> Co-authored-by: Patrick Miller <[email protected]> Co-authored-by: sovdee <[email protected]>
* Update CondIsCharged * Adds new test and updates effect - Also deletes old test * Fixes test - Wither skull wasn't spawning as I thought it was, changed to shoot effect * Apply suggestions from code review Co-authored-by: sovdee <[email protected]> * Suggestions --------- Co-authored-by: sovdee <[email protected]> Co-authored-by: Moderocky <[email protected]>
* Commit * Commit x2 * whoops * Suggested * One last change * Commit * Commit x2 * whoops * Suggested * One last change * change * yes?! * moved config * e * simplification * e * pickle please 🙏 * stupid IDE * aaaaaaaaaaaaaaaaaa * Changes, once again * chagne * deleted left other thing from a pr that i didnt make * e * done * Cleanup + Remove config --------- Co-authored-by: sovdee <[email protected]> Co-authored-by: Moderocky <[email protected]>
* Fix ExprBookAuthors performance Java 17 * Update src/main/java/ch/njol/skript/expressions/ExprBookAuthor.java Co-authored-by: Moderocky <[email protected]> * Update ExprBookAuthor.java * Update src/main/java/ch/njol/skript/expressions/ExprBookAuthor.java Co-authored-by: Patrick Miller <[email protected]> * Update src/main/java/ch/njol/skript/expressions/ExprBookAuthor.java Co-authored-by: sovdee <[email protected]> --------- Co-authored-by: Moderocky <[email protected]> Co-authored-by: Patrick Miller <[email protected]> Co-authored-by: sovdee <[email protected]>
#6912) * Add SecFilter, refactor Variable and Variables slightly to accomodate. * Messed up merge * realllly bad at merging * generify convertIfOldPlayer * use SectionNode#isEmpty() * small edits and implement variableIterator#remove() * Update src/main/java/ch/njol/skript/sections/SecFilter.java Co-authored-by: Patrick Miller <[email protected]> --------- Co-authored-by: Patrick Miller <[email protected]>
* Start work on new json docs generation * Add more to the JSON * Add sections * Add ID generation to JSON docs * Fix SkriptCommand errors * Add license header * Test duplicate checker * Existing IDs actually start at 2 * Address reviews * Remove test class * Address reviews * Address reviews * Try new way of generating IDs * Add license header * Fix nullable import * Add javadocs * Remove license headers * Address reviews * Address reviews * Address reviews Co-authored-by: _tud <[email protected]> * Address reviews * Address reviews --------- Co-authored-by: _tud <[email protected]>
Co-authored-by: sovdee <[email protected]>
* Multiple Random Numbers Support Took 26 minutes * Change pattern a wee bit. Took 2 minutes * Use parse tags instead of parse marks. Took 3 minutes * Requested Changes Took 15 minutes * Requested Changes Took 3 minutes * Apply suggestions from code review Co-authored-by: Ayham Al Ali <[email protected]> * Requested changes, added tests, bug fixes * Add check for literal 1 in isSingle * Use Integer instead of Number * forgot to delete the infinity tests * Update skript-aliases --------- Co-authored-by: sovdee <[email protected]> Co-authored-by: Ayham Al Ali <[email protected]> Co-authored-by: Moderocky <[email protected]>
* Add player chat completion suggestions * Add method exists check * Syntax adjustment and remove DataFlowIssue suppression * Docs improvements * Requested changes * Update src/main/java/ch/njol/skript/expressions/ExprPlayerChatCompletions.java Co-authored-by: sovdee <[email protected]> * Remove license header (#6684) * Java 17 ready --------- Co-authored-by: sovdee <[email protected]> Co-authored-by: Moderocky <[email protected]>
* Entity Potion Effect Event and event-values * Updated syntax for EvtEntityPotion, and readded the imports to EventValues. * Forgot to change e to event. * Added suggested changes :) * silly me * even sillier me (how did I manage to do that) * even sillier me (how did I manage to do that) * I don't get why the build is failing * BRO * Added type stuff, and made check a lot better * Changes * change * done * resolve conflict * Update src/main/java/ch/njol/skript/classes/data/BukkitClasses.java Co-authored-by: _tud <[email protected]> --------- Co-authored-by: sovdee <[email protected]> Co-authored-by: _tud <[email protected]>
* adds more entity potion causes and adds the type * Update default.lang * requested changes
* Fixes 5804-is-burning regression test - Previously used a wait to allow the pig's fire ticks to update, but waits aren't allowed in tests, so now just uses the ignite effect * Suggestions Co-authored-by: Efy <[email protected]> --------- Co-authored-by: Efy <[email protected]>
Fixes missing and/or warnings in ExprJoinSplit.sk and SecFilter.sk Co-authored-by: sovdee <[email protected]>
* Combine activity events and add unload event * Add global script events * Add EventRegister API * Javadoc improvements * Refactoring * Cleanup and additional refactoring * Add ScriptLoadEvent and ScriptInitEvent * Add PreScriptInitEvent This is an alternative for the now-deprecated Bukkit implementation * Minor javadoc corrections * Oopsie! add license headers * Requested formatting changes * Javadoc tweaks * Fix script load triggering * Remove license headers * Update class and method names * Formatting tweaks * Refactor event classes into appropriate locations The event classes have been refactored as inner classes of the classes they concern. This is to help accomdate future API changes (e.g. to ParserInstance, ScriptLoader, etc.) where the events may need to change. * Add Event parent class for type restriction --------- Co-authored-by: sovdee <[email protected]>
* Fix gradle temp * Start on displays * Save * More work * update gradle.build * stash * Progress so far * Finish bulk add * Finalize so far * Minimize adventure api * Fix static class check for not 1.19.4 * Fix missing display classinfo in pattern under 1.19.4 * Fix missing quaternions classinfo in pattern under 1.19.4 * Make sure test runs on only 1.19.4 * Address reviews and fix java 8 * Apply suggestions from code review Co-authored-by: sovdee <[email protected]> * Move classes * Apply changes * Move classes * Class rename * Update to Spigot's fix to TextAlignment * Return null when using w of vector * Update src/main/java/ch/njol/skript/expressions/ExprXYZComponent.java Co-authored-by: sovdee <[email protected]> * Use brackets * Add entity hiding * Add property init values for displays * 1.19.4 check * Add rotate syntaxes * Change example * Fix tests * shadow syntax missing * Change setting * Revert change * Update src/main/java/ch/njol/skript/conditions/CondCanSee.java * Fix imports from master merge * Get tests working * Add color reset * Update ExprXYZComponent.java * Update DisplayData.java * Update DisplayData.java * Change the package naming and merge item into its own class * Add versioning for hide effect * Add versioning for hide effect * Change tests * Update submodule skript-aliases * Fix tests * Fix tests * Fix tests * Apply suggestions from code review Co-authored-by: sovdee <[email protected]> * Apply changes * Add axis angle * Add axis angle * Update skript-aliases * fix merge error * Remove axisangle and only present quaternion to users * Add tests for basic quaternion features, cleanup and fix bugs with ExprXYZComponent, add tests Notable changes include changing float comparison behavior (needs discussion) and allowing multiple vectors/quaternions to be changed at once (will overwrite list indices, needs discussion) * Add ExprQuaternionAxisAngle and tests. Change float comparison to use dynamic epsilon value. * Temporary change to build.gradle for java 17 * Revert "Update src/main/java/ch/njol/skript/conditions/CondCanSee.java" This reverts commit 3da7b37. * Reverts entity hiding commits in favor of #6867 Shouldn't be in the scope of this PR. * version check for EQAA and displaydata cleanup * force 'rotation' in EQAA, fix classdef error, add itemtypes to block display entitydata pattern, add tests for display entitydata Not sure wh * Clean up colorOf, add tests, add alpha parameter to rgb function and color tostring, add rgba factory method + constructor Discussion needed surrounding type checking in colorof acceptChange * update Nullable annotations to jetbrains * Fix <1.19 errors (just for actions) and fix subtle issues with xyz component changer * rename package to bukkit * finish package refactor * Add tests for billboarding, rework display brightness and add tests, move joml utilities * tests + allow setting override to a single number * tests, cleanup, and bug fixes for glow override and hegiht/width * Interpolation and Shadow tests + correct reset value for shadow strength + guard against NaN and init * Tests and cleanup for teleportation duration, remove number from changers for tp duration and interpolation delay/duration * Transformation tests, cleanup, bugfixes. Removed unnecessary Joml utils class. * wrong version check for teleportation duration * Display view range tests, cleanup, finite guard, switch to Timespan#getAs * strip licenses * cleanup, tests for ItemDisplayTransfrom. Fixed default from being FIXED to NONE, changed lang for NONE to be no transform to avoid confusion * Remove ItemOfItemDisplay (redundant), cleanup/test the text display expressions. * small cleanup to textdisplayexprs and rename ItemFrameSlot to be more generic * rework ExprRotate * Completely rework rotation with new Rotator API. Add support for quaternions + displays in EffRotate * cleanup rotations - java 17 tests will fail until legacy tests are removed * Fix bugs with rotation, adds tests * Add drop shadow syntax and small cleanup of ExprTextOf * See through blocks syntax * Update ExprRotate.java * remove extra [a]'s * whoops! reverts a change to number accuracy config done for testing * Module-ize, package infos, remove mc version checks * rename display.expressions package, use for loops, move displaydata * display data improvements * requested changes 1 * update switches and remove banana * requested changes 2 * Apply suggestions from code review Co-authored-by: cheeezburga <[email protected]> * Apply suggestions from code review Co-authored-by: cheeezburga <[email protected]> * fix expritemofentity description * Apply suggestions from code review Co-authored-by: cheeezburga <[email protected]> * Update ExprDisplayHeightWidth.java * Apply suggestions from code review Co-authored-by: Patrick Miller <[email protected]> * requested changes 2 + rotator docs * requested changes * Apply suggestions from code review Co-authored-by: Patrick Miller <[email protected]> * requested changes! --------- Co-authored-by: sovdee <[email protected]> Co-authored-by: sovdee <[email protected]> Co-authored-by: Moderocky <[email protected]> Co-authored-by: cheeezburga <[email protected]> Co-authored-by: Patrick Miller <[email protected]>
* Added gamemode to CondIsInvulnerable.java Took 1 hour 13 minutes * Update src/main/java/ch/njol/skript/conditions/CondIsInvulnerable.java example fixed Co-authored-by: _tud <[email protected]> * Update src/main/java/ch/njol/skript/conditions/CondIsInvulnerable.java since* fixed Co-authored-by: _tud <[email protected]> * Update src/main/java/ch/njol/skript/conditions/CondIsInvulnerable.java only allowing entities and gamemodes* Co-authored-by: _tud <[email protected]> * Update src/main/java/ch/njol/skript/conditions/CondIsInvulnerable.java i literally forgot about curly brackets for a sec Co-authored-by: _tud <[email protected]> * Added methodExists check Took 10 minutes * checks if the method for gamemodes exists when registering the condition, fixed typo Took 9 minutes * all gamemodes over variable in example Co-authored-by: Patrick Miller <[email protected]> * tests Took 4 minutes * Refactor + space Took 5 minutes * Update src/test/skript/tests/syntaxes/conditions/CondIsInvulnerable.sk Co-authored-by: sovdee <[email protected]> * updated test Took 8 minutes * updated test Took 2 minutes * Update src/test/skript/tests/syntaxes/conditions/CondIsInvulnerable.sk Co-authored-by: sovdee <[email protected]> * Update src/test/skript/tests/syntaxes/conditions/CondIsInvulnerable.sk Co-authored-by: _tud <[email protected]> * Update CondIsInvulnerable.sk * Update CondIsInvulnerable.sk --------- Co-authored-by: _tud <[email protected]> Co-authored-by: sovdee <[email protected]> Co-authored-by: Patrick Miller <[email protected]>
--------- Co-authored-by: sovdee <[email protected]> Co-authored-by: Moderocky <[email protected]>
* Remove all old aliases and update skript-aliases to reference minimized branch. * fix spigot bugs and load-bearing performance issues this was so fuckin cursed * slightly less jank fix * rework alias parsing to allow __ block and __ item for appropriate materials * Update src/test/skript/tests/regressions/6707-spawn-fish-types.sk Co-authored-by: Patrick Miller <[email protected]> * Update Aliases.java * Clean up aliases file. Co-authored-by: Moderocky <[email protected]>
Remove underscore. --------- Co-authored-by: sovdee <[email protected]>
* ExprName - combine 3 registrations * ExprName - cleanup docs * ExprName - suggestion from sovde
) * Fix the burning time expression not returning super * Change the test name to match the issue * Update src/test/skript/tests/regressions/7373-burning-time-in-section.sk --------- Co-authored-by: Moderocky <[email protected]>
apparently 20 ticks wasn't enough Co-authored-by: Moderocky <[email protected]>
Update .gitmodules Co-authored-by: Moderocky <[email protected]>
fix Co-authored-by: Moderocky <[email protected]>
* Fix wrong unknown nullability on suppress warning enum * Update aliases * Requested changes * Revert long line field
* Since - support multiple lines * HTMLGenerator - add back empty since thingy * HTMLGenerator - remove some extra lines * HTMLGenerator - add "pre" to pattern * HTMLGenerator - jk jk * HTMLGenerator - fix "new" thingy in docs * HTMLGenerator - change useless matcher to false * Update src/main/java/ch/njol/skript/doc/HTMLGenerator.java Co-authored-by: Patrick Miller <[email protected]> * HTMLGenerator - only check the last value of a "since" --------- Co-authored-by: Patrick Miller <[email protected]>
…ime Errors. (#7383) Fix missing newline and handle syntaxes without documentation. Co-authored-by: Moderocky <[email protected]>
* SkriptCommand - fix test command again * SkriptCommandTabCompleter - fix differentiating between test and reload/etc
Support multiple commands in EvtCommand and add tests Co-authored-by: Moderocky <[email protected]>
* DocumentationIdProvider - fix collision checker always returning at least 1 * DocumentationIdProvider - fix typo * DocumentationIdProvider - change number 2 * DocumentationIdProvider - fix event ids
* EffAssert - fix null error when 'errorMsg' is not set * EffAssert - mark field nullable --------- Co-authored-by: Efnilite <[email protected]>
EffTestPluralClassInfos - remove from docs
* Testing. * I'll have the third part first, please. * Revert "Testing." This reverts commit 52c3b75.
Fix duplicate secspression logs Co-authored-by: Efnilite <[email protected]> Co-authored-by: Moderocky <[email protected]>
* init commit * fix incorrect order of typed args in getter * oops! --------- Co-authored-by: Moderocky <[email protected]>
* Add slightly more custom highlighting control * Removes RuntimeErrorProducer#toHighlight() - Highlighting defaults to null now, if error(String) is used * Removes toHighlight() override in SPE * Adds getRawExpr() method to SPE * Makes rawExpr field protected instead of having a public method to access it - Cheers Sovde --------- Co-authored-by: Moderocky <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
2.10
Targeting a 2.10.X version release
don't merge me !!
For pull requests that should not be merged due to some outstanding dispute, conflict or dependency.
feature-ready
A PR/issue that has been approved, tested and can be merged/closed in the next feature version.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is the 2.10 update commit.
Merge this on release day, and then create the release from master branch so it has the right signature.
Note: make sure to merge patch into feature before this.