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

[Fix]: Update DiffPatch and introduce testing #218

Merged
merged 8 commits into from
Jun 26, 2024

Conversation

marchermans
Copy link
Contributor

@marchermans marchermans commented Jun 25, 2024

Changes:

  • Updates DP to fix an issue with patch baking
  • Create E2E test framework and tests for testing platform module
    • Test patch generation
    • Test publication
    • Test userdev consumption
  • Add support for config caching in platform
  • Mark a bunch of initial TODOs for generalizing Platform better
  • Run parchment using conflict resolution mode
    • Add a prefix configuration option, which can be set by the user
    • Update to JST 43
    • Use the none checked variant

Note on the test setup:

Currently there is a massive build script needed to deal with publishing artifacts, this will likely need to be moved to NG Platform to make testing easier.

Fix issues with patch baking due to bugs in DP2
Add roundtrip testing for platform module
@neoforged-pr-publishing
Copy link

neoforged-pr-publishing bot commented Jun 25, 2024

  • Publish PR to GitHub Packages

Last commit published: 3b4c3ac7a19c4486115897e4e70b46cedee06922.

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 #218' // https://github.com/neoforged/NeoGradle/pull/218
        url 'https://prmaven.neoforged.net/NeoGradle/pr218'
        content {
            includeModule('net.neoforged.gradle', 'common')
            includeModule('net.neoforged.gradle', 'vanilla')
            includeModule('net.neoforged.gradle', 'test-utils')
            includeModule('net.neoforged.gradle.userdev', 'net.neoforged.gradle.userdev.gradle.plugin')
            includeModule('net.neoforged.gradle', 'utils')
            includeModule('net.neoforged.gradle', 'userdev')
            includeModule('net.neoforged.gradle', 'dsl-neoform')
            includeModule('net.neoforged.gradle', 'dsl-common')
            includeModule('net.neoforged.gradle.common', 'net.neoforged.gradle.common.gradle.plugin')
            includeModule('net.neoforged.gradle', 'dsl-mixin')
            includeModule('net.neoforged.gradle.platform', 'net.neoforged.gradle.platform.gradle.plugin')
            includeModule('net.neoforged.gradle', 'mixin')
            includeModule('net.neoforged.gradle.mixin', 'net.neoforged.gradle.mixin.gradle.plugin')
            includeModule('net.neoforged.gradle', 'dsl-vanilla')
            includeModule('net.neoforged.gradle', 'dsl-platform')
            includeModule('net.neoforged.gradle', 'neoform')
            includeModule('net.neoforged.gradle', 'dsl-userdev')
            includeModule('net.neoforged.gradle', 'platform')
            includeModule('net.neoforged.gradle.vanilla', 'net.neoforged.gradle.vanilla.gradle.plugin')
            includeModule('net.neoforged.gradle.neoform', 'net.neoforged.gradle.neoform.gradle.plugin')
        }
    }
}

@marchermans marchermans merged commit 2a8a833 into NG_7.0 Jun 26, 2024
106 checks passed
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.

2 participants