Skip to content

fix(quickstart): correct grammar in step 1 (review feedback) - #330

Merged
petermuessig merged 1 commit into
mainfrom
fix/quickstart-grammar-klattg
Aug 13, 2026
Merged

fix(quickstart): correct grammar in step 1 (review feedback)#330
petermuessig merged 1 commit into
mainfrom
fix/quickstart-grammar-klattg

Conversation

@petermuessig

Copy link
Copy Markdown
Contributor

Stacked on top of #329.

Addresses the grammar review comments from @KlattG on #328 (they landed after that PR was merged, so they were never applied):

  • packages/quickstart/steps/01/README.md — "To use install the required OpenUI5 libraries" → "To install the required OpenUI5 libraries" (removed the stray "use").
  • packages/quickstart/steps/01/README.md — "so we can use them in our project" → "so we can use it in our project" (singular — the tooling extension).

Both are verbatim applications of KlattG's suggestion comments. No other changes.

Note: this PR targets fix/tutorial-rendering-consistency (the #329 branch). Once #329 merges, GitHub will auto-retarget this to main.

Base automatically changed from fix/tutorial-rendering-consistency to main August 13, 2026 07:40
Address KlattG's review comments on #328:
- "To use install the required OpenUI5 libraries" -> "To install the
  required OpenUI5 libraries" (stray "use").
- "so we can use them in our project" -> "so we can use it in our
  project" (singular: the tooling extension).
@petermuessig
petermuessig force-pushed the fix/quickstart-grammar-klattg branch from 8688d8e to 7d85101 Compare August 13, 2026 11:46
@petermuessig
petermuessig merged commit 819cdd5 into main Aug 13, 2026
3 checks passed
@petermuessig
petermuessig deleted the fix/quickstart-grammar-klattg branch August 13, 2026 11:51
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.

3 participants