Skip to content

1f89a-2026.10.31-12.23.17-0400 - #3

Open
Fiaguhop137 wants to merge 3 commits into
Modcafe:mainfrom
Fiaguhop137:main
Open

Fiaguhop137 wants to merge 3 commits into
Modcafe:mainfrom
Fiaguhop137:main

Conversation

@Fiaguhop137

Copy link
Copy Markdown

No description provided.

@intensed-dev
intensed-dev self-requested a review October 31, 2026 16:55

@intensed-dev intensed-dev left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

  1. Please set a good name for your PR, so we know what it does
  2. Please add a Description, what your PR does and why we should merge it.

@intensed-dev
intensed-dev marked this pull request as draft October 31, 2026 16:57
@intensed-dev intensed-dev added missing information There's some Information missing, like a good title or description Unrelevant This PR doesn't seem relevant labels Oct 31, 2026
@Fiaguhop137
Fiaguhop137 marked this pull request as ready for review October 31, 2026 17:23
@Fiaguhop137

Copy link
Copy Markdown
Author

idk i fixed ur nullpointerexception

@intensed-dev

Copy link
Copy Markdown
Member

I will test the changes in game before merging. Alternatively, @ThatCuteOne could also do that if she's free.

@intensed-dev intensed-dev removed the Unrelevant This PR doesn't seem relevant label Oct 31, 2026
@Fiaguhop137

Copy link
Copy Markdown
Author

you wont see much its a bug fix not a feature add but okay

@intensed-dev

Copy link
Copy Markdown
Member

I'm just checking if everything works and try to understand what this code does in order to add more features later

@ThatCuteOne

ThatCuteOne commented Nov 1, 2026 •

Copy link
Copy Markdown
Member

you definitely need to start writing commit message that actually say what you changed instead of only the timestamp

Edit from @intensed-dev: Yes

@Fiaguhop137

Copy link
Copy Markdown
Author

i have this script thing i use to auto commit and i usually have a changelog.md

@Fiaguhop137 Fiaguhop137 closed this Nov 1, 2026
@Fiaguhop137 Fiaguhop137 reopened this Nov 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

missing information There's some Information missing, like a good title or description

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants