Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Expose some methods #8

Closed
wants to merge 2 commits into from
Closed

Expose some methods #8

wants to merge 2 commits into from

Conversation

Matyrobbrt
Copy link
Member

Expose some fields and methods as API to allow JST to apply the ATs to the source.

@Matyrobbrt Matyrobbrt added the enhancement New feature or request label Apr 14, 2024
@neoforged-pr-publishing
Copy link

neoforged-pr-publishing bot commented Apr 14, 2024

  • Publish PR to GitHub Packages

Last commit published: bc5de18c6b7a63de4bc652e5b79fd9fd81b0a71c.

PR Publishing

The artifacts published by this PR:

Repository Declaration

In order to use the artifacts published by the PR, add the following repository to your buildscript:

repositories {
    maven {
        name 'Maven for PR #8' // https://github.com/neoforged/AccessTransformers/pull/8
        url 'https://prmaven.neoforged.net/AccessTransformers/pr8'
        content {
            includeModule('net.neoforged', 'accesstransformers')
            includeModule('net.neoforged.accesstransformers', 'at-cli')
            includeModule('net.neoforged.accesstransformers', 'at-modlauncher')
        }
    }
}

@lukebemish
Copy link

After some discussion in discord, I've messed around with the new non-antlr parser in #6 to expose (I believe) all the relevant stuff as a separate at-parser artifact, de-coupling the structures AT files are read into from the logic for applying their transformations.

@Matyrobbrt Matyrobbrt closed this Apr 14, 2024
@Matyrobbrt Matyrobbrt deleted the expose-methods branch April 14, 2024 21:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants