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

Ongoing Minor Issue List (Sponge Edition) #23

Open
Deamon5550 opened this issue May 4, 2015 · 2 comments
Open

Ongoing Minor Issue List (Sponge Edition) #23

Deamon5550 opened this issue May 4, 2015 · 2 comments
Labels
priority: low Low priority status: accepted a confirmation that this is either an issue caused by us or something that should be added

Comments

@Deamon5550
Copy link
Contributor

Deamon5550 commented May 4, 2015

SpongeAPI | Sponge

Checkstyle issues should not be reported here as those can be more accurately determined by simply running the checkstyle tool. Examples of good issues to report here are misnamed methods/fields, poorly worded javadocs, formatting issues.

As minor things are resolved, comments will be deleted.

@Deamon5550 Deamon5550 added status: accepted a confirmation that this is either an issue caused by us or something that should be added help wanted priority: low Low priority labels May 4, 2015
@Deamon5550 Deamon5550 changed the title Ongoing Minor Issue List Ongoing Minor Issue List (SpongeCommon Edition) May 4, 2015
kashike pushed a commit that referenced this issue Nov 29, 2016
kashike pushed a commit that referenced this issue Oct 1, 2017
@SpongePowered SpongePowered deleted a comment from ryantheleach Oct 1, 2017
@SpongePowered SpongePowered deleted a comment from Lunaphied Oct 1, 2017
@SpongePowered SpongePowered deleted a comment from JBYoshi Dec 22, 2017
@SpongePowered SpongePowered deleted a comment from JBYoshi Dec 22, 2017
@SpongePowered SpongePowered deleted a comment from JBYoshi Dec 22, 2017
@SpongePowered SpongePowered deleted a comment from kashike Dec 22, 2017
@SpongePowered SpongePowered deleted a comment from liach Dec 22, 2017
@SpongePowered SpongePowered deleted a comment from gabizou Feb 3, 2018
@SpongePowered SpongePowered deleted a comment from liach Jun 14, 2018
@phit phit pinned this issue May 25, 2019
@Cybermaxke Cybermaxke unpinned this issue Jun 3, 2019
@Cybermaxke Cybermaxke pinned this issue Jun 3, 2019
@SpongePowered SpongePowered deleted a comment from LXGaming Aug 1, 2019
@SpongePowered SpongePowered deleted a comment from LordRalex Aug 1, 2019
@SpongePowered SpongePowered deleted a comment from liach Aug 1, 2019
@SpongePowered SpongePowered deleted a comment from ejm Jun 18, 2021
@SpongePowered SpongePowered deleted a comment from Wyrdix Jun 18, 2021
@SpongePowered SpongePowered deleted a comment from LogicFan Jun 18, 2021
@SpongePowered SpongePowered deleted a comment from kashike Jun 18, 2021
@SpongePowered SpongePowered deleted a comment from ejm Jun 18, 2021
@SpongePowered SpongePowered deleted a comment from JBYoshi Jun 18, 2021
@SpongePowered SpongePowered deleted a comment from JBYoshi Jun 18, 2021
@SpongePowered SpongePowered deleted a comment from Lignium Jun 18, 2021
@SpongePowered SpongePowered deleted a comment from Lignium Jun 18, 2021
@SpongePowered SpongePowered deleted a comment from DrZoddiak Jun 18, 2021
@SpongePowered SpongePowered deleted a comment from JBYoshi Jun 18, 2021
@SpongePowered SpongePowered deleted a comment from JBYoshi Jun 18, 2021
@SpongePowered SpongePowered deleted a comment from JBYoshi Jun 18, 2021
@SpongePowered SpongePowered deleted a comment from kashike Jun 18, 2021
@Zidane Zidane changed the title Ongoing Minor Issue List (SpongeCommon Edition) Ongoing Minor Issue List (Sponge Edition) Jun 18, 2021
@ejm
Copy link

ejm commented Jul 5, 2021

@GuusLieben
Copy link

There is a typo in the method name for setIsUnbrekable (should be setIsUnbreakable) at https://github.com/SpongePowered/Sponge/blob/api-8/src/main/java/org/spongepowered/common/data/provider/item/stack/ItemStackData.java#L207

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority: low Low priority status: accepted a confirmation that this is either an issue caused by us or something that should be added
Projects
None yet
Development

No branches or pull requests

3 participants