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

Fixed a mistake in the FilenameUtils.concat()'s Javadoc about an absolute path. #96

Merged
merged 3 commits into from Sep 11, 2019

Conversation

yuji-k64613
Copy link
Contributor

ng
/foo/a/ + ../bar --> foo/bar

ok
/foo/a/ + ../bar --> /foo/bar

/foo/a/ + ../bar     -->   /foo/bar
@coveralls
Copy link

coveralls commented Sep 10, 2019

Coverage Status

Coverage remained the same at 89.308% when pulling dcf1cdf on yuji-k64613:IO-626-FilenameUtils into d463fa0 on apache:master.

Copy link

@Stzx Stzx left a comment

Choose a reason for hiding this comment

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

This is just a minor change and it doesn't seem to need to update changes.xml.

@yuji-k64613
Copy link
Contributor Author

The Travis CI build failed, what should I do?

@Stzx
Copy link

Stzx commented Sep 10, 2019

This should be a fake failure.

@garydgregory
Copy link
Member

This could be a random failure and certainly should not be affected by a Javadoc changes.

@yuji-k64613 : If you want to be credited, please update src/changes/changes.xml, otherwise, please say if you do not here.

@yuji-k64613 yuji-k64613 changed the title Fixed a mistake in the FilenameUtils.concat()'s Javadoc about a absolute path. Fixed a mistake in the FilenameUtils.concat()'s Javadoc about an absolute path. Sep 11, 2019
@@ -134,6 +134,9 @@ The <action> type attribute can be add,update,fix,remove.
<action issue="IO-625" dev="ggregory" type="fix" due-to="Mikko Maunu">
Corrected misleading exception message for FileUtils.copyDirectoryToDirectory.
</action>
<action issue="IO-626" dev="Yuji Konishi" type="fix" due-to="Yuji Konishi">
Copy link

Choose a reason for hiding this comment

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

Normally the dev is a committer, and due-to is the PR contributor name/alias.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sorry.

I thought that a committer and the PR contributor are me.
Which is my mistake, a committer or the PR contributor?

@garydgregory garydgregory merged commit 11020bc into apache:master Sep 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants