Skip to content

Fix RDoc::Attr#add_alias handling of aliased attribute accessor#1605

Merged
tompng merged 1 commit intoruby:masterfrom
extern-c:fix-add-alias-for-attribute-accessor
Feb 9, 2026
Merged

Fix RDoc::Attr#add_alias handling of aliased attribute accessor#1605
tompng merged 1 commit intoruby:masterfrom
extern-c:fix-add-alias-for-attribute-accessor

Conversation

@extern-c
Copy link
Contributor

@extern-c extern-c commented Feb 8, 2026

Ensure aliased attr_accessor attributes have the correct access type ('R' or 'W') instead of always being 'RW'.

For more details on the issue being addressed, see #1286 (comment).

@extern-c extern-c requested a deployment to fork-preview-protection February 8, 2026 20:41 — with GitHub Actions Waiting
@extern-c extern-c requested a deployment to fork-preview-protection February 8, 2026 20:49 — with GitHub Actions Waiting
Ensure aliased attributes get the correct access type ('R' or 'W') instead of always being marked as 'RW'.
@extern-c extern-c force-pushed the fix-add-alias-for-attribute-accessor branch from b12db40 to 82ec898 Compare February 9, 2026 02:32
@extern-c extern-c temporarily deployed to fork-preview-protection February 9, 2026 02:33 — with GitHub Actions Inactive
Copy link
Member

@tompng tompng left a comment

Choose a reason for hiding this comment

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

Looks good 👍

@matzbot
Copy link
Collaborator

matzbot commented Feb 9, 2026

🚀 Preview deployment available at: https://0f0fb6f2.rdoc-6cd.pages.dev (commit: 82ec898)

@tompng tompng enabled auto-merge (squash) February 9, 2026 19:17
@tompng tompng disabled auto-merge February 9, 2026 19:28
@tompng tompng merged commit c6ecfe5 into ruby:master Feb 9, 2026
32 checks passed
@tompng tompng added the bug label Feb 9, 2026
@extern-c extern-c deleted the fix-add-alias-for-attribute-accessor branch February 16, 2026 03:41
tompng pushed a commit that referenced this pull request Feb 17, 2026
…ypes (#1612)

This is a continuation of #1605. 

The original PR ensured that aliased attr_accessor attributes have the
correct access type ('R' or 'W') instead of always being 'RW'.

This PR adds test cases to verify that aliased reader and writer
`RDoc::Attr`s are created with the correct access type ('R' or 'W').
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants