Skip to content

Readd support for client and server sides and update to 26.1-snapshot-11#29

Open
coehlrich wants to merge 9 commits intoneoforged:mainfrom
coehlrich:client-and-server
Open

Readd support for client and server sides and update to 26.1-snapshot-11#29
coehlrich wants to merge 9 commits intoneoforged:mainfrom
coehlrich:client-and-server

Conversation

@coehlrich
Copy link
Contributor

Client and server sides shouldn't have been removed in the first place since other projects may be using them.

@neoforged-pr-publishing
Copy link

neoforged-pr-publishing bot commented Mar 3, 2026

  • Publish PR to GitHub Packages

Last commit published: deffdb6ea755ed70b70476ac64877817f163e36f - version: 26.1-pre-1-snapshot

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 #29" // https://github.com/neoforged/NeoForm/pull/29
        url = uri("https://prmaven.neoforged.net/NeoForm/pr29")
        content {
            includeModule("net.neoforged", "neoform")
        }
    }
}

@shartte
Copy link
Contributor

shartte commented Mar 3, 2026

Client and server sides shouldn't have been removed in the first place since other projects may be using them.

We generally don't do more work because others might be using it.

@Technici4n
Copy link
Member

Seconded. We need a clear motivation for why it is needed. As far as I know, nobody complained that we removed it. ;)

@coehlrich coehlrich force-pushed the client-and-server branch from 15bbb36 to d79926c Compare March 4, 2026 01:49
@neoforged-automation neoforged-automation bot added the needs rebase This Pull Request needs to be rebased before being merged label Mar 11, 2026
@neoforged-automation
Copy link

@coehlrich, this pull request has conflicts, please resolve them for this PR to move forward.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs rebase This Pull Request needs to be rebased before being merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants