Skip to content

docs(changelog): backfill v0.2.0–v0.3.3 and add PR #34 entry#35

Merged
kwent merged 2 commits into
masterfrom
features/review-pr-34
Jun 9, 2026
Merged

docs(changelog): backfill v0.2.0–v0.3.3 and add PR #34 entry#35
kwent merged 2 commits into
masterfrom
features/review-pr-34

Conversation

@kwent

@kwent kwent commented Jun 9, 2026

Copy link
Copy Markdown
Member

Summary

Test plan

  • Verify changelog formatting renders correctly on GitHub

@kwent kwent merged commit 11edbb0 into master Jun 9, 2026
5 checks passed
@greptile-apps

greptile-apps Bot commented Jun 9, 2026

Copy link
Copy Markdown

Greptile Summary

This PR backfills six missing changelog versions (v0.2.0–v0.3.3) by redistributing content that was previously lumped in [Unreleased], and adds the PR #34 on-call filter fixes as a new [Unreleased] entry.

  • One fix bullet (Windows test compatibility — USERPROFILE alongside HOME) present in the original [Unreleased] block was not assigned to any versioned section and is now absent from the file entirely.
  • The [Unreleased] section lists ### Fixed before ### Added, reversing the standard Keep a Changelog ordering (Added → Changed → Deprecated → Removed → Fixed → Security).
  • All compare links at the bottom are correctly updated to chain from v0.1.5 through v0.3.3 to HEAD.

Confidence Score: 4/5

Documentation-only change with no code impact; safe to merge after recovering the dropped Windows fix entry.

The backfill is well-structured and the compare links are correct, but one fix entry was silently dropped during the reorganisation, leaving a gap in the historical record.

CHANGELOG.md — verify the Windows test compatibility fix is placed in the correct version and the [Unreleased] section order is corrected.

Important Files Changed

Filename Overview
CHANGELOG.md Backfills 6 missing version entries (v0.2.0–v0.3.3) and adds PR #34 to [Unreleased]; one fix entry (Windows test compatibility) from the old [Unreleased] block was dropped during reorganisation, and the [Unreleased] section lists Fixed before Added against Keep a Changelog convention.
Prompt To Fix All With AI
Fix the following 2 code review issues. Work through them one at a time, proposing concise fixes.

---

### Issue 1 of 2
CHANGELOG.md:63-65
**Missing fix dropped during backfill**

The previous `[Unreleased]` section contained `- Windows test compatibility (\`USERPROFILE\` alongside \`HOME\`)` under `### Fixed`, but this entry was not moved to any of the new versioned sections during the backfill — it is now entirely absent from the changelog. Based on its proximity to the oncall changes it likely belongs in `v0.2.0` (or `v0.2.1`) under `### Fixed`.

### Issue 2 of 2
CHANGELOG.md:8-16
The `[Unreleased]` section lists `### Fixed` before `### Added`, reversing the conventional Keep a Changelog order (Added → Changed → Deprecated → Removed → Fixed → Security).

```suggestion
## [Unreleased]

### Added
- Name-based filtering for `oncall shifts` and `oncall who` (`--schedule`, `--service`, `--user`, `--team` flags)
- Team/group filtering via `--team-id` / `--team` flags

### Fixed
- Fix plural filter param names for `/v1/oncalls` endpoint (`filter[schedule_ids]`, `filter[service_ids]`, etc.)
- URL-encode filter values across all list endpoints
```

Reviews (1): Last reviewed commit: "docs(changelog): add PR #34 on-call filt..." | Re-trigger Greptile

Comment thread CHANGELOG.md
Comment on lines 63 to 65
### Fixed
- Fix `oncall schedules` 404 error (use correct `/v1/schedules` endpoint)
- Windows test compatibility (`USERPROFILE` alongside `HOME`)
- Correct env var name in docs from `ROOTLY_API_TOKEN` to `ROOTLY_API_KEY`

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Missing fix dropped during backfill

The previous [Unreleased] section contained - Windows test compatibility (\USERPROFILE` alongside `HOME`)under### Fixed, but this entry was not moved to any of the new versioned sections during the backfill — it is now entirely absent from the changelog. Based on its proximity to the oncall changes it likely belongs in v0.2.0(orv0.2.1) under ### Fixed`.

Prompt To Fix With AI
This is a comment left during a code review.
Path: CHANGELOG.md
Line: 63-65

Comment:
**Missing fix dropped during backfill**

The previous `[Unreleased]` section contained `- Windows test compatibility (\`USERPROFILE\` alongside \`HOME\`)` under `### Fixed`, but this entry was not moved to any of the new versioned sections during the backfill — it is now entirely absent from the changelog. Based on its proximity to the oncall changes it likely belongs in `v0.2.0` (or `v0.2.1`) under `### Fixed`.

How can I resolve this? If you propose a fix, please make it concise.

Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!

Comment thread CHANGELOG.md
Comment on lines 8 to +16
## [Unreleased]

### Fixed
- Fix plural filter param names for `/v1/oncalls` endpoint (`filter[schedule_ids]`, `filter[service_ids]`, etc.)
- URL-encode filter values across all list endpoints

### Added
- Name-based filtering for `oncall shifts` and `oncall who` (`--schedule`, `--service`, `--user`, `--team` flags)
- Team/group filtering via `--team-id` / `--team` flags

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 The [Unreleased] section lists ### Fixed before ### Added, reversing the conventional Keep a Changelog order (Added → Changed → Deprecated → Removed → Fixed → Security).

Suggested change
## [Unreleased]
### Fixed
- Fix plural filter param names for `/v1/oncalls` endpoint (`filter[schedule_ids]`, `filter[service_ids]`, etc.)
- URL-encode filter values across all list endpoints
### Added
- Name-based filtering for `oncall shifts` and `oncall who` (`--schedule`, `--service`, `--user`, `--team` flags)
- Team/group filtering via `--team-id` / `--team` flags
## [Unreleased]
### Added
- Name-based filtering for `oncall shifts` and `oncall who` (`--schedule`, `--service`, `--user`, `--team` flags)
- Team/group filtering via `--team-id` / `--team` flags
### Fixed
- Fix plural filter param names for `/v1/oncalls` endpoint (`filter[schedule_ids]`, `filter[service_ids]`, etc.)
- URL-encode filter values across all list endpoints
Prompt To Fix With AI
This is a comment left during a code review.
Path: CHANGELOG.md
Line: 8-16

Comment:
The `[Unreleased]` section lists `### Fixed` before `### Added`, reversing the conventional Keep a Changelog order (Added → Changed → Deprecated → Removed → Fixed → Security).

```suggestion
## [Unreleased]

### Added
- Name-based filtering for `oncall shifts` and `oncall who` (`--schedule`, `--service`, `--user`, `--team` flags)
- Team/group filtering via `--team-id` / `--team` flags

### Fixed
- Fix plural filter param names for `/v1/oncalls` endpoint (`filter[schedule_ids]`, `filter[service_ids]`, etc.)
- URL-encode filter values across all list endpoints
```

How can I resolve this? If you propose a fix, please make it concise.

Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!

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.

1 participant