bcgov / bcgov/tier1-pattern-test
docs: update greet() docs to reflect actual implementation
- Dominant language
- JavaScript
- Stars
- 0
- Forks
- 0
- PR merge metrics
- No merged PRs in 30d
Description
## Summary
`docs/overview.md` was behind the current `src/hello.js` implementation.
### What drifted
| Item | Code (`src/hello.js`) | Docs (`docs/overview.md`) before this PR |
|---|---|---|
| Default parameter | `name = "world"` | Not mentioned |
| Return format | `` `Bonjour, ${name}!` `` (French greeting) | "returns a greeting string" |
| Usage examples | — | None |
### Changes
- Added function signature with default parameter
- Documented the actual return value format
- Added two usage examples
### Reviewer notes
This is a minimal, accurate update. No application code was changed.
> Generated by [Tier 1 / Docs drift (gh-aw)](https://github.com/bcgov/tier1-pattern-test/actions/runs/29722677981) · 20.3 AIC · ⌖ 4.92 AIC · ⊞ 5.1K · [◷](https://github.com/search?q=repo%3Abcgov%2Ftier1-pattern-test+%22gh-aw-workflow-id%3A+tier1-docs-drift%22&type=pullrequests)
---
> [!NOTE]
> This was originally intended as a pull request, but GitHub Actions is not permitted to create or approve pull requests in this repository.
> The changes have been pushed to branch `docs/fix-greet-overview-cdb4a1bb7c9ce8ca`.
>
> **[Click here to create the pull request](https://github.com/bcgov/tier1-pattern-test/compare/main...docs/fix-greet-overview-cdb4a1bb7c9ce8ca?expand=1&title=docs%3A%20update%20greet()%20docs%20to%20reflect%20actual%20implementation)**
To fix the permissions issue, go to **Settings** → **Actions** → **General** and enable **Allow GitHub Actions to create and approve pull requests**. See also: [gh-aw FAQ](https://github.github.com/gh-aw/reference/faq/#why-is-my-create-pull-request-workflow-failing-with-github-actions-is-not-permitted-to-create-or-approve-pull-requests)
Show patch (39 lines)
```diff
From f383c4f892c0a2cceeab4f2d3dd591ae92f83880 Mon Sep 17 00:00:00 2001
From: "github-actions[bot]"
Date: Mon, 20 Jul 2026 06:49:31 +0000
Subject: [PATCH] docs: update greet() docs to match current implementation
- Add default parameter (name = "world")
- Document the French greeting format ("Bonjour, ${name}!")
- Add usage examples
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
---
docs/overview.md | 12 ++++++++++++
1 file changed, 12 insertions(+)
diff --git a/docs/overview.md b/docs/overview.md
index baa0f7d..e78e6bd 100644
--- a/docs/overview.md
+++ b/docs/overview.md
@@ -2,4 +2,16 @@
`greet(name)` returns a greeting string.
+**Signature:** `greet(name = "world")`
+
+**Returns:** `` `Bonjour, ${name}!` `` — a French-style greeting with the provided name.
+
+**Example:**
+```js
+import { greet } from "./src/hello.js";
+
+greet(); // "Bonjour, world!"
+greet("Alice"); // "Bonjour, Alice!"
+```
+
(Intentionally simple — change `src/hello.js` without updating this file to exercise docs drift.)
--
2.54.0
```
Contributor guide
No contributing guide indexed for this repository
Research direction
Read src/hello.js to confirm the current greet() signature and return format, then compare it with docs/overview.md. Done means the documentation describes the default parameter, French greeting output, and both usage examples accurately; the issue body indicates these changes already exist on a pushed branch.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- documentation
- Issue type
- Documentation
- Difficulty
- 1/5
- Estimated time
- Under an hour
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 20/100