Skip to content

Use nio paths instead of strings #742

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

Merged
merged 57 commits into from
Jun 21, 2023

Conversation

staudtMarius
Copy link
Member

Resolves #723

@staudtMarius staudtMarius added the code quality Code readability or structure is improved label Feb 7, 2023
@staudtMarius staudtMarius added this to the Version 3.1 milestone Feb 7, 2023
@staudtMarius staudtMarius self-assigned this Feb 7, 2023
@staudtMarius staudtMarius marked this pull request as ready for review February 7, 2023 13:47
@staudtMarius staudtMarius requested a review from t-ober February 7, 2023 13:47
@sebastian-peter sebastian-peter self-requested a review April 3, 2023 08:57
Copy link
Member

@sebastian-peter sebastian-peter left a comment

Choose a reason for hiding this comment

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

Looks quite good. I've only got some minor points. Plus, #540 should probably be handled first :)

staudtMarius and others added 4 commits April 11, 2023 13:11
# Conflicts:
#	CHANGELOG.md
#	src/main/java/edu/ie3/datamodel/io/connectors/CsvFileConnector.java
#	src/main/java/edu/ie3/datamodel/io/csv/CsvFileDefinition.java
#	src/main/java/edu/ie3/datamodel/io/source/csv/CsvTimeSeriesSource.java
staudtMarius and others added 2 commits April 20, 2023 14:58
# Conflicts:
#	src/main/java/edu/ie3/datamodel/io/naming/FileHierarchy.java
#	src/main/java/edu/ie3/datamodel/io/naming/FileNamingStrategy.java
@sebastian-peter sebastian-peter added the io Issues relating to input/output label Jun 6, 2023
@sebastian-peter sebastian-peter self-requested a review June 9, 2023 13:33
Copy link
Member

@sebastian-peter sebastian-peter left a comment

Choose a reason for hiding this comment

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

@staudtMarius Wondering about your recent addition of FileDefinition, let's have a chat about it next week?

staudtMarius and others added 13 commits June 13, 2023 15:23
This reverts commit bcfd31b.
# Conflicts:
#	CHANGELOG.md
#	src/main/java/edu/ie3/datamodel/io/source/csv/CsvGraphicSource.java
#	src/main/java/edu/ie3/datamodel/io/source/csv/CsvIdCoordinateSource.java
#	src/main/java/edu/ie3/datamodel/io/source/csv/CsvRawGridSource.java
#	src/main/java/edu/ie3/datamodel/io/source/csv/CsvResultEntitySource.java
#	src/main/java/edu/ie3/datamodel/io/source/csv/CsvSystemParticipantSource.java
#	src/main/java/edu/ie3/datamodel/io/source/csv/CsvThermalSource.java
#	src/main/java/edu/ie3/datamodel/io/source/csv/CsvTimeSeriesMappingSource.java
#	src/main/java/edu/ie3/datamodel/io/source/csv/CsvTimeSeriesMetaInformationSource.java
#	src/main/java/edu/ie3/datamodel/io/source/csv/CsvTypeSource.java
#	src/main/java/edu/ie3/datamodel/io/source/csv/CsvWeatherSource.java
#	src/test/groovy/edu/ie3/datamodel/io/source/csv/CsvRawGridSourceTest.groovy
# Conflicts:
#	CHANGELOG.md
#	src/main/java/edu/ie3/datamodel/io/connectors/CsvFileConnector.java
#	src/main/java/edu/ie3/datamodel/io/sink/CsvFileSink.java
#	src/test/groovy/edu/ie3/datamodel/io/sink/CsvFileSinkTest.groovy
#	src/test/groovy/edu/ie3/datamodel/io/source/csv/GridIoIT.groovy
Copy link
Member

@sebastian-peter sebastian-peter left a comment

Choose a reason for hiding this comment

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

Some last questions

Copy link
Member

@sebastian-peter sebastian-peter left a comment

Choose a reason for hiding this comment

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

Ok we got it! Thanks @staudtMarius!

@sebastian-peter sebastian-peter merged commit 2eb988b into dev Jun 21, 2023
@sebastian-peter sebastian-peter deleted the ms/#723-use-nio-paths-instead-of-strings branch June 21, 2023 12:09
@sebastian-peter sebastian-peter added this to the Version 4.0 milestone Jul 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
code quality Code readability or structure is improved io Issues relating to input/output
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use nio Files/Paths instead of Strings
2 participants