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

Updated index.md #36845

Merged
merged 3 commits into from
Nov 18, 2024
Merged

Updated index.md #36845

merged 3 commits into from
Nov 18, 2024

Conversation

Vipulagarwal-2000
Copy link
Contributor

@Vipulagarwal-2000 Vipulagarwal-2000 commented Nov 18, 2024

fixed the sentence structure from "not learning a new tool" to "learning a new tool" which sound correct.

Description

This is the original sentence.

With this knowledge, you can focus on your contributions and not learn a new tool at the same time.

This is the modified sentence

With this knowledge, you can focus on your contributions and learn a new tool at the same time.

Motivation

This is my first open-source contribution. I am excited about doing this. Second, this is a simple typo, but it will make the sentence clear.

Additional details

Here is the page link:
page link

fixed the sentence structure from "not learning a new tool" to "learning a new tool" which sound correct.
@Vipulagarwal-2000 Vipulagarwal-2000 requested a review from a team as a code owner November 18, 2024 13:27
@Vipulagarwal-2000 Vipulagarwal-2000 requested review from hamishwillee and removed request for a team November 18, 2024 13:27
@github-actions github-actions bot added Content:Meta Content in the meta docs size/xs [PR only] 0-5 LoC changed labels Nov 18, 2024
Copy link
Contributor

github-actions bot commented Nov 18, 2024

Preview URLs

External URLs (1)

URL: /en-US/docs/MDN/Community/Contributing/Getting_started
Title: Getting started with MDN Web Docs

(comment last updated: 2024-11-18 16:33:11)

@github-actions github-actions bot added size/s [PR only] 6-50 LoC changed and removed size/xs [PR only] 0-5 LoC changed labels Nov 18, 2024
Co-authored-by: Brian Thomas Smith <brian@smith.berlin>
Copy link
Contributor Author

@Vipulagarwal-2000 Vipulagarwal-2000 left a comment

Choose a reason for hiding this comment

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

yeah now it look better and perfect.

Copy link
Member

@bsmth bsmth left a comment

Choose a reason for hiding this comment

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

Super, thanks a lot!

@bsmth bsmth merged commit 6232eb0 into mdn:main Nov 18, 2024
8 checks passed
@Vipulagarwal-2000 Vipulagarwal-2000 deleted the Vipulagarwal-2000-patch-1 branch November 19, 2024 03:58
wbamberg added a commit to wbamberg/content that referenced this pull request Nov 27, 2024
* origin/xss-guide: (284 commits)
  Add information on default entryPoint property values (mdn#36633)
  Bump husky from 9.1.6 to 9.1.7 (mdn#36863)
  fix(performance): Typo '50ms seconds' (mdn#36861)
  Add spec_url & add note for bcd for `<frequency>` and `<frequency-percentage>` (mdn#36848)
  addresses 36583: summary icon styles (mdn#36691)
  Remove "simple" part 3: change to "basic"  (mdn#36762)
  the default option of a select (mdn#36658)
  docs(css): Add support for `<string>` in `syntax` descriptor of @Property at-rule (mdn#36655)
  Fix parameter syntax for `Navigation.updateCurrentEntry()` (mdn#36852)
  Update CSP source expression reference (mdn#36792)
  chore(http): Refresh headers docs (d-k) (mdn#36075)
  chore(http): Refresh headers r-s (mdn#36590)
  Updated index.md (mdn#36845)
  fix : wrong method name (mdn#36843)
  Remove all redirects to other locales (mdn#36811)
  fix typos (mdn#36837)
  docs: update Accept-Charset status (mdn#36822)
  updateSelection: make more intuitive (mdn#36834)
  updateText: Remove false information (mdn#36832)
  DOMRect instance properties (mdn#36704)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content:Meta Content in the meta docs size/s [PR only] 6-50 LoC changed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants