Summary
As of #20786 we should now enforce those changes in PRs going forward.
Basic example
def initialize(info = {})
super(
update_info(...
'Arch' => 'unix',
'Targets' => [ ... /* define a unix target here */ ... ]
) )
The usage of top-level arch, platform etc. can be removed, as the information will get merged from the target.
Motivation
This has been called out during module hacking sessions, and we ask contributors to include these changes as blockers before landing modules.
This would save our and the contributor cycles.
Summary
As of #20786 we should now enforce those changes in PRs going forward.
Basic example
The usage of top-level arch, platform etc. can be removed, as the information will get merged from the target.
Motivation
This has been called out during module hacking sessions, and we ask contributors to include these changes as blockers before landing modules.
This would save our and the contributor cycles.