Skip to content

make it modern - #3

Merged
plicease merged 21 commits into
mainfrom
graham/make-it-modern
Oct 6, 2026
Merged

plicease merged 21 commits into
mainfrom
graham/make-it-modern

Conversation

@plicease

@plicease plicease commented Oct 5, 2026

Copy link
Copy Markdown
Member

No description provided.

plicease and others added 21 commits October 5, 2026 17:31
Use [@author::Plicease] like File-Which: drop Makefile.PL, MANIFEST,
META.yml and README; fold Event.pod into Event.pm; add ABSTRACT/VERSION
markers; add static workflow and author tooling config.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Expand tabs using 4-column tab stops.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Correct typos, fix ETIMEOUT -> ETIMEDOUT in ie_connect_failed docs,
add stopwords to author.yml, and regenerate README.md.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Document the event loop functions in IO::Event and the
IO::Event::Callback constructor, and mark internal backends and
callback dispatch methods as private in author.yml.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Don't write to the sync pipe after the last test, since the child
has already exited.  Also remove unreachable code in ie_input.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Rename to descriptive lexicals.  No interface change.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Annotate the existing uses line by line so the policy still applies
to new code.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Replace FindBin in the test wrappers with require './t/...', and
annotate the AnyEvent uses line by line so the policy still applies
to new code.  Regenerate README.md and t/00_diag.t.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The runtime backend selection in IO::Event::import still assigns
@isa directly and is annotated for Perl::Critic.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Remove the dead trailing 1; from the test templates, which always
exit before reaching it, and annotate the exit after confess in
ie_do_invoke.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Use strict and warnings at the top, use parent instead of @isa,
use three-argument open, and remove the unreachable trailing 1;.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
It is defined in IO/Event.pm, but the prereq scanner picks it up
from use parent -norequire.  Regenerate t/00_diag.t.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Read into lexicals instead of $_ in while loops to fix the last
WhileDiamondDefaultAssignment violations, and add xt/author/critic.t
(checking lib, t/*.t and t/*.tt, but not the generated t/00_diag.t).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@plicease
plicease merged commit 3857f5a into main Oct 6, 2026
21 checks passed
@plicease
plicease deleted the graham/make-it-modern branch October 6, 2026 01: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.

1 participant