PaperMC / PaperMC/paperweight

Option to fail on missing/failed AT application

Open
#344 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Kotlin
Stars
317
Forks
76
Avg merge
1d 12h
Merged PRs (30d)
5

Description

Expose a configuration option in paperweight-core/patcher to fail if the application of access transformers via JST fails.
For this, --access-transformer-validation=error should be passed to JST for a representative exit code.

This should help during updates when a lot of AT targets become invalid as they were renamed/removed.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start in paperweight-core/patcher and trace how JST is invoked and how configuration options are exposed. Add the access-transformer validation option so JST receives --access-transformer-validation=error when selected, and verify that failed or missing AT applications produce a representative failure.

Written by the indexing model from the issue text.

Assessment

Tech stack
kotlin
Domain
build-system, tooling
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.