Skip to content

fix(briefcombat.lic): v1.0.1 fix in @instance variable location#2219

Merged
mrhoribu merged 2 commits intomasterfrom
fix/briefcombat-v1.0.1
Jan 29, 2026
Merged

fix(briefcombat.lic): v1.0.1 fix in @instance variable location#2219
mrhoribu merged 2 commits intomasterfrom
fix/briefcombat-v1.0.1

Conversation

@mrhoribu
Copy link
Contributor

@mrhoribu mrhoribu commented Jan 29, 2026

Important

Fixes bug by moving @instance variable to instance-level in Engine class of briefcombat.lic, with version update to 1.0.1.

  • Behavior:
    • Fixes bug by moving @instance variable from class-level to instance-level in initialize method of Engine class in briefcombat.lic.
    • Updates version to 1.0.1 to reflect bug fix.
  • Debugging:
    • Adds additional debug output in compress_combat() for regex matching.
    • Modifies debug output in brief() to show full line instead of truncated version.

This description was created by Ellipsis for f2165f0. You can customize this summary. It will automatically update as commits are pushed.

Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Important

Looks good to me! 👍

Reviewed everything up to f2165f0 in 13 seconds. Click for details.
  • Reviewed 386 lines of code in 1 files
  • Skipped 0 files when reviewing.
  • Skipped posting 0 draft comments. View those below.
  • Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.

Workflow ID: wflow_pYWUYKksMOYnl9Jc

You can customize Ellipsis by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.

@mrhoribu mrhoribu merged commit 323aa45 into master Jan 29, 2026
3 of 4 checks passed
@mrhoribu mrhoribu deleted the fix/briefcombat-v1.0.1 branch January 29, 2026 22:09
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.

1 participant