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

Triple PR for #111 #112 #113 #114

Merged
merged 4 commits into from
Oct 6, 2023
Merged

Triple PR for #111 #112 #113 #114

merged 4 commits into from
Oct 6, 2023

Conversation

tribbloid
Copy link
Collaborator

Submitting Uber-patch is never a good idea, I'll try to submit smaller patches in the future.

All changes are reflected in test cases update (mostly in BasicSpec/__direct and the new files), they should be easy to read

idea plugin of gradle is now defined for all modules

Refined types Tuple types &  now shows multi-row format if too long

add 3 test cases for the above change

splitters (e.g. comma) won't take an line in multi-row display

fix a bug where multiple indexed Based can flush out each other

simplify & optimise formatting of Based in both flat & broken formats

add settings VtypeDetail and VtypeDiffsDetail

VtypeDiffsDetail now extended to cover =:= and <:<, guarded by 2 more test cases

add a shortcut to disable plugin and all options in each test case

add a debug option to show original message in foundReqMsg

Plugin option "all" splitted into "enabled" and "enableAll"
@github-actions
Copy link

github-actions bot commented Oct 4, 2023

Test Report (2.13.9)

97 tests  +19   97 ✔️ +19   14s ⏱️ +2s
15 suites +  2     0 💤 ±  0 
15 files   +  2     0 ±  0 

Results for commit b57adf6. ± Comparison against base commit 3d6fe23.

This pull request removes 4 and adds 23 tests. Note that renamed tests count towards both.
splain.builtin.MaxRefinedSpec ‑ original
splain.builtin.VerboseTreeSpec ‑ original
splain.plugin.VTypeReductionSpec ‑ foundReqVsImplicit
splain.plugin.VTypeReductionSpec ‑ original
splain.builtin.BasicSpec ‑ #111 LongRefined
splain.builtin.BasicSpec ‑ #111 LongTuple
splain.builtin.BasicSpec ‑ #111 foundReqLongTuple
splain.builtin.BasicSpec ‑ #34 compoundDiff
splain.builtin.BasicSpec ‑ LongArg
splain.builtin.BasicSpec ‑ foundReqSameSymbol
splain.builtin.BasicXSource3Spec ‑ #111 LongRefined
splain.builtin.BasicXSource3Spec ‑ #111 LongTuple
splain.builtin.BasicXSource3Spec ‑ #111 foundReqLongTuple
splain.builtin.BasicXSource3Spec ‑ #34 compoundDiff
…

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Oct 4, 2023

Test Report (2.13.10)

97 tests  +19   97 ✔️ +19   16s ⏱️ +5s
15 suites +  2     0 💤 ±  0 
15 files   +  2     0 ±  0 

Results for commit b57adf6. ± Comparison against base commit 3d6fe23.

This pull request removes 4 and adds 23 tests. Note that renamed tests count towards both.
splain.builtin.MaxRefinedSpec ‑ original
splain.builtin.VerboseTreeSpec ‑ original
splain.plugin.VTypeReductionSpec ‑ foundReqVsImplicit
splain.plugin.VTypeReductionSpec ‑ original
splain.builtin.BasicSpec ‑ #111 LongRefined
splain.builtin.BasicSpec ‑ #111 LongTuple
splain.builtin.BasicSpec ‑ #111 foundReqLongTuple
splain.builtin.BasicSpec ‑ #34 compoundDiff
splain.builtin.BasicSpec ‑ LongArg
splain.builtin.BasicSpec ‑ foundReqSameSymbol
splain.builtin.BasicXSource3Spec ‑ #111 LongRefined
splain.builtin.BasicXSource3Spec ‑ #111 LongTuple
splain.builtin.BasicXSource3Spec ‑ #111 foundReqLongTuple
splain.builtin.BasicXSource3Spec ‑ #34 compoundDiff
…

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Oct 4, 2023

Test Report (2.13.11)

97 tests  +19   97 ✔️ +19   12s ⏱️ -1s
15 suites +  2     0 💤 ±  0 
15 files   +  2     0 ±  0 

Results for commit b57adf6. ± Comparison against base commit 3d6fe23.

This pull request removes 4 and adds 23 tests. Note that renamed tests count towards both.
splain.builtin.MaxRefinedSpec ‑ original
splain.builtin.VerboseTreeSpec ‑ original
splain.plugin.VTypeReductionSpec ‑ foundReqVsImplicit
splain.plugin.VTypeReductionSpec ‑ original
splain.builtin.BasicSpec ‑ #111 LongRefined
splain.builtin.BasicSpec ‑ #111 LongTuple
splain.builtin.BasicSpec ‑ #111 foundReqLongTuple
splain.builtin.BasicSpec ‑ #34 compoundDiff
splain.builtin.BasicSpec ‑ LongArg
splain.builtin.BasicSpec ‑ foundReqSameSymbol
splain.builtin.BasicXSource3Spec ‑ #111 LongRefined
splain.builtin.BasicXSource3Spec ‑ #111 LongTuple
splain.builtin.BasicXSource3Spec ‑ #111 foundReqLongTuple
splain.builtin.BasicXSource3Spec ‑ #34 compoundDiff
…

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Oct 4, 2023

Test Report (2.13.12)

97 tests  +19   97 ✔️ +19   13s ⏱️ -1s
15 suites +  2     0 💤 ±  0 
15 files   +  2     0 ±  0 

Results for commit b57adf6. ± Comparison against base commit 3d6fe23.

This pull request removes 4 and adds 23 tests. Note that renamed tests count towards both.
splain.builtin.MaxRefinedSpec ‑ original
splain.builtin.VerboseTreeSpec ‑ original
splain.plugin.VTypeReductionSpec ‑ foundReqVsImplicit
splain.plugin.VTypeReductionSpec ‑ original
splain.builtin.BasicSpec ‑ #111 LongRefined
splain.builtin.BasicSpec ‑ #111 LongTuple
splain.builtin.BasicSpec ‑ #111 foundReqLongTuple
splain.builtin.BasicSpec ‑ #34 compoundDiff
splain.builtin.BasicSpec ‑ LongArg
splain.builtin.BasicSpec ‑ foundReqSameSymbol
splain.builtin.BasicXSource3Spec ‑ #111 LongRefined
splain.builtin.BasicXSource3Spec ‑ #111 LongTuple
splain.builtin.BasicXSource3Spec ‑ #111 foundReqLongTuple
splain.builtin.BasicXSource3Spec ‑ #34 compoundDiff
…

♻️ This comment has been updated with latest results.

…not required

some test cases are prefixed by their github issue number
@github-actions
Copy link

github-actions bot commented Oct 4, 2023

Test Report (2.13.8)

97 tests  +19   97 ✔️ +19   13s ⏱️ -1s
15 suites +  2     0 💤 ±  0 
15 files   +  2     0 ±  0 

Results for commit b57adf6. ± Comparison against base commit 3d6fe23.

This pull request removes 4 and adds 23 tests. Note that renamed tests count towards both.
splain.builtin.MaxRefinedSpec ‑ original
splain.builtin.VerboseTreeSpec ‑ original
splain.plugin.VTypeReductionSpec ‑ foundReqVsImplicit
splain.plugin.VTypeReductionSpec ‑ original
splain.builtin.BasicSpec ‑ #111 LongRefined
splain.builtin.BasicSpec ‑ #111 LongTuple
splain.builtin.BasicSpec ‑ #111 foundReqLongTuple
splain.builtin.BasicSpec ‑ #34 compoundDiff
splain.builtin.BasicSpec ‑ LongArg
splain.builtin.BasicSpec ‑ foundReqSameSymbol
splain.builtin.BasicXSource3Spec ‑ #111 LongRefined
splain.builtin.BasicXSource3Spec ‑ #111 LongTuple
splain.builtin.BasicXSource3Spec ‑ #111 foundReqLongTuple
splain.builtin.BasicXSource3Spec ‑ #34 compoundDiff
…

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Oct 4, 2023

Test Report (2.13.7)

97 tests  +19   97 ✔️ +19   16s ⏱️ +5s
15 suites +  2     0 💤 ±  0 
15 files   +  2     0 ±  0 

Results for commit b57adf6. ± Comparison against base commit 3d6fe23.

This pull request removes 4 and adds 23 tests. Note that renamed tests count towards both.
splain.builtin.MaxRefinedSpec ‑ original
splain.builtin.VerboseTreeSpec ‑ original
splain.plugin.VTypeReductionSpec ‑ foundReqVsImplicit
splain.plugin.VTypeReductionSpec ‑ original
splain.builtin.BasicSpec ‑ #111 LongRefined
splain.builtin.BasicSpec ‑ #111 LongTuple
splain.builtin.BasicSpec ‑ #111 foundReqLongTuple
splain.builtin.BasicSpec ‑ #34 compoundDiff
splain.builtin.BasicSpec ‑ LongArg
splain.builtin.BasicSpec ‑ foundReqSameSymbol
splain.builtin.BasicXSource3Spec ‑ #111 LongRefined
splain.builtin.BasicXSource3Spec ‑ #111 LongTuple
splain.builtin.BasicXSource3Spec ‑ #111 foundReqLongTuple
splain.builtin.BasicXSource3Spec ‑ #34 compoundDiff
…

♻️ This comment has been updated with latest results.

@tribbloid tribbloid merged commit 140e6e9 into master Oct 6, 2023
19 checks passed
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