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

Feature/checkstyle #31

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Feature/checkstyle #31

wants to merge 5 commits into from

Conversation

Zanganito
Copy link
Collaborator

No description provided.

@Zanganito
Copy link
Collaborator Author

Adds an changes

@Zanganito
Copy link
Collaborator Author

Please, can you checks this new changes? thanks.

@r3back
Copy link
Owner

r3back commented Aug 8, 2023

Sorry I can't man

@Zanganito
Copy link
Collaborator Author

it's okay :(

}), 0, 20);
private void whenStart() {
Bukkit.getScheduler().runTaskTimer(this, () -> Bukkit.getServer().getOnlinePlayers().
forEach(player -> {
Copy link
Owner

Choose a reason for hiding this comment

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

Cuando separas tenes que bajar el punto también, te quedaría .forEach en la línea de abajo

* Makes an auctions service
*
* @return {@link AuctionService}
*/
Copy link
Owner

Choose a reason for hiding this comment

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

En este caso más que un Makes quedaría mejor un Retrieves

UserService service();
Plugin plugin();
/**
* Makes a config files
Copy link
Owner

Choose a reason for hiding this comment

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

Usar Retrieves en todos estos

@@ -17,8 +20,21 @@ public final class AuctionCategory extends OkaeriConfig {
private GUIDisplayItem displayInfo;
private List<CategoryFilter> filters;

/**
* Makes an auction category
Copy link
Owner

Choose a reason for hiding this comment

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

En este caso no iría el makes porque es el constructor, reemplaza el esa línea por Default Constructor

*
* @param id Id
* @param displayName Display Name
* @param materials {@link XMaterial}
Copy link
Owner

Choose a reason for hiding this comment

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

Usar List of

* @param displayName Display Name
* @param materials {@link XMaterial}
* @param displayInfo {@link GUIDisplayItem}
* @param filters {@link CategoryFilter}
Copy link
Owner

Choose a reason for hiding this comment

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

Usar List of

import lombok.Builder;
import lombok.Getter;

/**
* Utility class dor category filter
Copy link
Owner

Choose a reason for hiding this comment

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

No es una clase de utilidad.

@Getter
public final class CategoryFilter extends OkaeriConfig {
/**
* Adds filter type
Copy link
Owner

Choose a reason for hiding this comment

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

Este comentario no va acá

private final FilterType type;
private final String nbtKey;

/**
* Represents a filter type
Copy link
Owner

Choose a reason for hiding this comment

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

Default Constructor

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.

None yet

2 participants