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

Change for with no lead #3190

Draft
wants to merge 12 commits into
base: beta
Choose a base branch
from
Draft

Change for with no lead #3190

wants to merge 12 commits into from

Conversation

AuroZed
Copy link

@AuroZed AuroZed commented Jan 9, 2025

Dependencies

What

A few changes and tries that doesnt really work in the end :D

@github-actions github-actions bot added the Wrong Title/Changelog There is an error in the title or changelog label Jan 9, 2025
Copy link

github-actions bot commented Jan 9, 2025

I have detected some issues with your pull request:

Body issues:
Author is not set in text: + Cool new feature. - your_name_here
Extra info is not filled out in text: * Extra info.
Author is not set in text: + Improved cool feature. - your_name_here
Extra info is not filled out in text: * Extra info.
Author is not set in text: + Fixed cool feature. - your_name_here
Extra info is not filled out in text: * Extra info.
Author is not set in text: + Something technical you changed in the backend. - your_name_here
Extra info is not filled out in text: * Extra info.
Author is not set in text: + Removed cool feature. - your_name_here
Extra info is not filled out in text: * Extra info.

Title issues:
PR title does not match the expected format of 'Category: Title'

Please fix these issues. For the correct format, refer to the pull request template.

@github-actions github-actions bot added the Detekt Has detekt problem label Jan 9, 2025
Copy link

github-actions bot commented Jan 9, 2025

edited, so that i dont need to scroll so much anymore ~hannibal

One or more Detekt Failures were detected:

Copy link

github-actions bot commented Jan 9, 2025

I have detected some issues with your pull request:

Body issues:
No changes detected in this pull request

Please fix these issues. For the correct format, refer to the pull request template.

@CalMWolfs
Copy link
Collaborator

hey if you want to discuss any of this or need some help on anything feel free to ask in the #coding channel in the skyhanni discord

Copy link

github-actions bot commented Jan 10, 2025

Again, i dont want to scroll so much ~hannibal2

One or more Detekt Failures were detected:

@hannibal002 hannibal002 added the Waiting on Dependency PR waiting for a dependency pr to be merged label Jan 10, 2025
Copy link

I have detected some issues with your pull request:

Body issues:
No changes detected in this pull request

Please fix these issues. For the correct format, refer to the pull request template.

Copy link
Owner

@hannibal002 hannibal002 left a comment

Choose a reason for hiding this comment

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

Given the amount of still commented out code i make this a draft pr.
Also please add the changelog in the pr description - see the template

@@ -17,7 +17,7 @@ public class TooltipTweaksConfig {
)
@ConfigEditorBoolean
@FeatureToggle
public boolean compactToolTooltips = false;
Copy link
Owner

Choose a reason for hiding this comment

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

unrelated change, please revert

import io.github.notenoughupdates.moulconfig.annotations.ConfigOption;
import org.lwjgl.input.Keyboard;

public class BreakDownConfig {
Copy link
Owner

Choose a reason for hiding this comment

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

please add the prefix mining to the class name, and lowercase the D in breakdown

@hannibal002 hannibal002 marked this pull request as draft January 10, 2025 23:09
Copy link

One or more Detekt Failures were detected:

@github-actions github-actions bot added the Merge Conflicts There are open merge conflicts with the beta branch. label Jan 10, 2025
Copy link

This pull request has conflicts with the base branch "beta". Please resolve those so we can test out your changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Detekt Has detekt problem Merge Conflicts There are open merge conflicts with the beta branch. Waiting on Dependency PR waiting for a dependency pr to be merged Wrong Title/Changelog There is an error in the title or changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants