Skip to content
This repository has been archived by the owner on Feb 27, 2024. It is now read-only.

Ongoing Minor Issue List (SpongeForge Edition) #263

Open
1 task
Deamon5550 opened this issue May 4, 2015 · 3 comments
Open
1 task

Ongoing Minor Issue List (SpongeForge Edition) #263

Deamon5550 opened this issue May 4, 2015 · 3 comments
Labels
priority: low Low priority status: accepted reproduction with minimal mods available

Comments

@Deamon5550
Copy link
Contributor

Deamon5550 commented May 4, 2015

SpongeAPI | SpongeCommon | SpongeForge | SpongeVanilla

A new and improved Minor issue list for Beta. This list will be handled a little differently in order to make it much clearer whether a reported issue has been noted and/or fixed. Minor issues reported here will be added to the main comment and the reporting comment will be deleted. 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, minor logical errors (like that something could be using a java8 stream).

Minor Issue List

  • Discover Minor Issues
@Deamon5550 Deamon5550 added help wanted status: accepted reproduction with minimal mods available priority: low Low priority labels May 4, 2015
@Deamon5550 Deamon5550 changed the title Ongoing Minor Issue List Ongoing Minor Issue List (Sponge Edition) May 4, 2015
@Zidane Zidane changed the title Ongoing Minor Issue List (Sponge Edition) Ongoing Minor Issue List (SpongeForge Edition) Jan 1, 2016
@liach

This comment has been minimized.

liach referenced this issue Jul 14, 2018
…performance overall.

Not likely to cause issues, but neighbor notifications are likely to be much faster as well.

Signed-off-by: Gabriel Harris-Rouquette <[email protected]>
@phit phit pinned this issue May 25, 2019
@LordRalex
Copy link

This should be just referring to SpongeLaunch.getAdditionalPluginsDir() which has the same logic

@JBYoshi
Copy link
Member

JBYoshi commented Jun 22, 2019

// This can be called before Sponge is initialied, so use this hack

Typo in "initialied"; missing a Z.

@gabizou gabizou unpinned this issue Jun 29, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
priority: low Low priority status: accepted reproduction with minimal mods available
Projects
None yet
Development

No branches or pull requests

4 participants