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

Update introduction.md in International Calling Connoisseur #2917

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

kadrisamuel
Copy link

Removed "studentScores", which seems to be there by mistake.

pull request


Reviewer Resources:

Track Policies

Removed "studentScores", which seems to be there by mistake.
@kadrisamuel kadrisamuel changed the title Update introduction.md Update introduction.md in International Calling Connoisseur Feb 26, 2025
Copy link
Member

@kahgoh kahgoh left a comment

Choose a reason for hiding this comment

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

Thanks for taking the time to raise this PR! Your suggested change looks good!

While reviewing, I noticed there were additional problems in the same area and I was wondering if we could fix them 🙏? I left comments to guide you through them.

@@ -56,7 +56,7 @@ The [size][map-size-javadoc] method returns the number of entries.
fruitPrices.size(); // Returns 2
```

You can use the [keys] or [values] methods to obtain the keys or the values in a Map as a Set or collection respectively.studentScores
You can use the [keys] or [values] methods to obtain the keys or the values in a Map as a Set or collection respectively.

```java
fruitPrices.keys(); // Returns "apple" and "pear" in a set
Copy link
Member

Choose a reason for hiding this comment

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

As mentioned in the other comment, should be keySet

Suggested change
fruitPrices.keys(); // Returns "apple" and "pear" in a set
fruitPrices.keySet(); // Returns "apple" and "pear" in a set

Could you also please check the following files for the same problem:

@@ -56,7 +56,7 @@ The [size][map-size-javadoc] method returns the number of entries.
fruitPrices.size(); // Returns 2
```

You can use the [keys] or [values] methods to obtain the keys or the values in a Map as a Set or collection respectively.studentScores
You can use the [keys] or [values] methods to obtain the keys or the values in a Map as a Set or collection respectively.
Copy link
Member

Choose a reason for hiding this comment

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

I noticed there are other issues here:

  • The [keys] and [values] are supposed to be links, but they are missing the URLs.
  • Even then, [keys] should be [keySet].

To make it easier, I've tried adding suggestions where I can:

Suggested change
You can use the [keys] or [values] methods to obtain the keys or the values in a Map as a Set or collection respectively.
You can use the [keySet][map-keyset-javadoc] or [values][map-values-javadoc] methods to obtain the keys or the values in a Map as a Set or collection respectively.

Will also need to add the following lines to the bottom of this file (I can't add seem to add these as suggestions):

map-keyset-javadoc: https://docs.oracle.com/en/java/javase/21/docs/api/java.base/java/util/Map.html#keySet()
map-values-javadoc: https://docs.oracle.com/en/java/javase/21/docs/api/java.base/java/util/Map.html#values()

Could you also please check the following files for the same problem:

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