docs(changelog): scrub profile-field noise [skip-runtime-e2e]#171
Merged
saurabhjain1592 merged 1 commit intomainfrom May 8, 2026
Merged
docs(changelog): scrub profile-field noise [skip-runtime-e2e]#171saurabhjain1592 merged 1 commit intomainfrom
saurabhjain1592 merged 1 commit intomainfrom
Conversation
The profile field added in #2013 / SDK PR #161 was removed in PR #170 before any v8.0.0 tag shipped. Customer-facing CHANGELOG should reflect what users experience, not internal flaps. Net effect of this scrub: - Delete the [8.0.1] section entirely. - Scrub `profile` mentions from the [8.0.0] section. - Revert version bump 8.0.1 → 8.0.0 in pom.xml + 2 example poms. Code state unchanged — main has the post-#170 telemetry surface (no profile field, no env-var read). Signed-off-by: Saurabh Jain <saurabhjain1592@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Customer-facing CHANGELOGs document what users experience. The
profilefield added in PR #161 was removed in PR #170 before any v8.0.0 tag shipped — customers never installed code that had this field. The "Removed: profile field" entry I added in PR #170 referenced internal noise.This PR:
[8.0.1]CHANGELOG section entirely.profilementions from the[8.0.0]section.8.0.1→8.0.0inpom.xml+ 2 example poms.Code state unchanged.
Why
User direction: "This never made it to any customer so why it should be in changelog when we removed it already before the release."
Skip-runtime-e2e justification
Touches `CHANGELOG.md` + `pom.xml`. While
.xmlfiles aren't in the auto-skip extension list, the change is documentation-only (version-revert + CHANGELOG edit) — no code behavior changes, no runtime behavior to verify. Runtime proof for the actual fix landed in #170.Linked