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

Improvements and Fixes #15

Open
wants to merge 16 commits into
base: master
Choose a base branch
from

Conversation

Wolf2323
Copy link

@Wolf2323 Wolf2323 commented Apr 16, 2024

The fork contains multiple improvements and fixes, that i needed.

@Wolf2323 Wolf2323 marked this pull request as draft April 16, 2024 13:14
Wolf2323 and others added 13 commits April 22, 2024 16:02
Add our repository and the required distributionManagement config, drop the maven-compiler-plugin config that's now inherited.
…main/{java,resources}

With this, we don't need the sourceDirectory override and custom resources config in the POM any longer.
All of that can be imported from the POM.
This is a single-module project, no need for a subdir, and this gets rid of the duplicated .gitignore, too.
Our MavenSetup currently expects the POM in the project root.
To avoid reformatting all whitespace, instead a custom EditorConfig specifies 4-space indent for Java files instead of our tabstop-based indentation.
Likewise, a CheckStyle suppression disables our custom Indentation rule.
…attributes

So that files are checked out with Unix-style line endings on Windows as well.
… warnings

Integrated via #1

Merge branch 'TASK-4714/build/MavenSetup'
@inkarkat inkarkat deleted the branch PowerNetworkHub:master July 24, 2024 13:59
@inkarkat inkarkat deleted the master branch July 24, 2024 13:59
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.

Rotation does not always rotate correctly Add reload command to get updates from file
2 participants