Skip to content

[pull] master from williamfiset:master#122

Merged
pull[bot] merged 8 commits intoUncodedtech:masterfrom
williamfiset:master
Mar 7, 2026
Merged

[pull] master from williamfiset:master#122
pull[bot] merged 8 commits intoUncodedtech:masterfrom
williamfiset:master

Conversation

@pull
Copy link

@pull pull bot commented Mar 7, 2026

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

williamfiset and others added 8 commits March 7, 2026 09:40
- Fix removeAt decrementing capacity (corrupted capacity tracking)
- Fix add() off-by-one in resize check (len+1>=capacity → len==capacity)
- Add bounds checks to get, set, and removeAt
- Make arr and len private (accessible via get/set/size)
- Remove commented-out code, fix string concatenation in toString
- Add 28 dedicated tests covering all operations and edge cases

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
)

Fix IntStack missing bounds checks on peek/pop (empty stack) and push
(overflow), add file-level headers with Big-O complexity to all files,
replace fully qualified class names with proper imports in ListStack
and IntStack, add @OverRide annotations, and remove commented-out code.

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
* Refactor Trie: add docs, fix naming, remove dead code

Add file-level header with use cases and Big-O complexity, add Javadoc
to all public methods, replace FQN with proper imports, rename snake_case
variables to camelCase, remove dead recursive clear(Node) method, remove
useless null assignments in delete(), inline addChild wrapper, and clean
up test imports.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* Change insert() to return true if the word already exists

Previously insert() returned true if the key contained a prefix already
in the trie. Now it returns true only if the exact word was already
inserted, which is a more intuitive and standard trie API.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Add file-level header with use cases and O(α(n)) complexity, add Javadoc
to public methods, fix package declaration order, simplify unify() to
avoid redundant find calls, and remove commented-out import in test.

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
* Fix README URL checker: upgrade Ruby, add bundler cache, allow 429s

Upgrade Ruby from 3.0 to 3.2, use bundler caching to speed up CI,
and allow HTTP 429 (rate limit) responses to reduce flaky failures.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* Revert bundler-cache, install awesome_bot directly

bundler-cache requires a Gemfile which awesome_bot doesn't use.
Install the gem globally instead and call it directly.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
- Delete build/ directory from tracking (193 generated files) and
  add it to .gitignore
- Delete .github/workflows/gradle.yml CI workflow
- Remove Gradle ecosystem from .github/dependabot.yml
- Replace ./gradlew run comments with bazel run in 22 source files
- Replace gradle test comments with bazel test in 14 test files

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@pull pull bot locked and limited conversation to collaborators Mar 7, 2026
@pull pull bot added the ⤵️ pull label Mar 7, 2026
@pull pull bot merged commit 4f8a72b into Uncodedtech:master Mar 7, 2026
2 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant