Skip to content

fix typo erroring->erring#201

Closed
EruDev wants to merge 5 commits intoapache:masterfrom
EruDev:master
Closed

fix typo erroring->erring#201
EruDev wants to merge 5 commits intoapache:masterfrom
EruDev:master

Conversation

@EruDev
Copy link
Copy Markdown
Contributor

@EruDev EruDev commented Dec 21, 2020

No description provided.

@EruDev EruDev changed the title fix type erroring->erring fix typo erroring->erring Dec 21, 2020
Copy link
Copy Markdown
Member

@kinow kinow left a comment

Choose a reason for hiding this comment

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

👍

@EruDev
Copy link
Copy Markdown
Contributor Author

EruDev commented Dec 21, 2020

@kinow I submitted another coomit, an invalid link

//-----------------------------------------------------------------------
/**
* Returns the {@link #collection} field cast to a {@link Bag}.
* Returns the {@link Collection} field cast to a {@link Bag}.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

That's a link to the collection variable of the parent class.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

sorry..I revert AbstractBagTest

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

No problem, thanks @EruDev . Could you squash your commits too? For a small change, it's preferable to have a single (or very few) commit. Please.

Thanks!

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I close this PR and resubmit.

@EruDev EruDev closed this Dec 21, 2020
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.

2 participants