forked from discordjs/discord.js
-
Notifications
You must be signed in to change notification settings - Fork 0
Doc/coverage measurement improvement #27
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
Merged
Merged
Conversation
This file contains hidden or 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
….com/DD2480-Group22-2025/DD2480-discord.js into doc/coverage-measurement-improvement
….com/DD2480-Group22-2025/DD2480-discord.js into doc/coverage-measurement-improvement
….com/DD2480-Group22-2025/DD2480-discord.js into doc/coverage-measurement-improvement
peschwartz
approved these changes
Feb 18, 2025
peschwartz
added a commit
that referenced
this pull request
Feb 18, 2025
feat: refactor thread patch reduces cyclomatic complexity in patch function in thread. #9 fix: moved files around Doc/coverage measurement improvement (#27) * docs: branchcoverage tool wip #13 * docs: improves #13 * docs: refactord guild.js patch function for #9 * Small fix #5 * docs: finish #13 * Revert "Small fix #5" This reverts commit 8513b3f. * docs: improved refactor for guild.js patch file #9 --------- Co-authored-by: Klara Lindemalm <[email protected]>
carltestar
added a commit
that referenced
this pull request
Feb 20, 2025
* fix: Do not omit falsy default values (discordjs#10755) * fix(docs): fix default falsy values being omitted * fix(docs): swap defaultValue check to avoid negated condition * fix: fix pr by removing everything it added and committing something entirely different --------- Co-authored-by: almeidx <[email protected]> doc/setup initial documentation files Update issue templates * docs: onboarding written * feat: lizard analysis of code complexity Update issue templates fix: update functions docs: add guidelines * fix: update functions * doc: answers task 3.4 part 1 (#21) * Elaborates on onboarding document (#22) * docs: elaborates on onboarding document #2 * Update onboarding.md Says tests instead of tasks Closes #2 --------- Co-authored-by: Jacmol <[email protected]> Update complexity.md * docs: added coverage measurement documentation * docs: update documentation for refactoring feat: refactor thread patch reduces cyclomatic complexity in patch function in thread. #9 fix: moved files around Doc/coverage measurement improvement (#27) * docs: branchcoverage tool wip #13 * docs: improves #13 * docs: refactord guild.js patch function for #9 * Small fix #5 * docs: finish #13 * Revert "Small fix #5" This reverts commit 8513b3f. * docs: improved refactor for guild.js patch file #9 --------- Co-authored-by: Klara Lindemalm <[email protected]> * docs: added documentation for refactor #26 (#28) Update complexity.md * feat: branch coverage and refactoring * fix: Updated log message for patch.test.js --------- Co-authored-by: Amgelo563 <[email protected]> Co-authored-by: phoebe <[email protected]> Co-authored-by: Samuel Söderberg <[email protected]> Co-authored-by: Klara Lindemalm <[email protected]> Co-authored-by: Phoebe Schwartz <[email protected]> Co-authored-by: Carltestar <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Closes #13 and fixes #26