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

[WFCORE-5393] Remove the wildlfy-elytron-auth-server-deprecated dependency module to no longer rely on deprecated classes #5377

Closed
wants to merge 1 commit into from

Conversation

sudeshnas93
Copy link
Contributor

@wildfly-ci
Copy link

Core -> Full Integration Build 12198 outcome was FAILURE using a merge of b482d40
Summary: Compilation error: Compiler (new) Build time: 00:00:39

@wildfly-ci
Copy link

Core -> WildFly Preview Integration Build 12126 outcome was FAILURE using a merge of b482d40
Summary: Compilation error: Compiler (new) Build time: 00:00:45

@wildfly-ci
Copy link

Core -> Full Integration Build 12008 outcome was FAILURE using a merge of b482d40
Summary: Compilation error: Compiler (new) Build time: 00:00:57

Copy link
Contributor

@darranl darranl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is going to be a bigger task than just a package rename.

These classes are advertised as capabilities, we also have external feature packs / extensions that work with these types that can be broken if we just change them all without some form of compatibility mode.

…ncy module to no longer rely on deprecated classes
@wildfly-ci
Copy link

Core -> Full Integration Build 12199 outcome was FAILURE using a merge of 2c30e3e
Summary: Compilation error: Compiler Build time: 00:00:24

@wildfly-ci
Copy link

Core -> Full Integration Build 12009 outcome was FAILURE using a merge of 2c30e3e
Summary: Compilation error: Compiler Build time: 00:00:35

@wildfly-ci
Copy link

Core -> WildFly Preview Integration Build 12127 outcome was FAILURE using a merge of 2c30e3e
Summary: Compilation error: Compiler Build time: 00:00:34

@darranl
Copy link
Contributor

darranl commented Feb 14, 2023

This is going to be a bigger task than just a package rename.

These classes are advertised as capabilities, we also have external feature packs / extensions that work with these types that can be broken if we just change them all without some form of compatibility mode.

The compilation failures in WildFly start to illustrate this problem.

@sudeshnas93
Copy link
Contributor Author

This PR can only be merged after merging this jboss-remoting PR jboss-remoting/jboss-remoting#286

@darranl darranl added the hold Do not merge this PR label Feb 14, 2023
@github-actions
Copy link

github-actions bot commented Apr 1, 2023

There has been no activity on this PR for 45 days. It will be auto-closed after 90 days.

@github-actions github-actions bot added the Stale label Apr 1, 2023
@wildfly-ci
Copy link

Core -> Full Integration Build 12355 outcome was FAILURE using a merge of 2c30e3e
Summary: Compilation error: Compiler (new) Build time: 00:00:36

@wildfly-ci
Copy link

Core -> Full Integration Build 12157 outcome was FAILURE using a merge of 2c30e3e
Summary: Compilation error: Compiler (new) Build time: 00:00:37

@wildfly-ci
Copy link

Core -> WildFly Preview Integration Build 12269 outcome was FAILURE using a merge of 2c30e3e
Summary: Compilation error: Compiler (new) Build time: 00:00:39

@github-actions github-actions bot removed the Stale label Apr 2, 2023
@github-actions
Copy link

There has been no activity on this PR for 45 days. It will be auto-closed after 90 days.

@github-actions github-actions bot added the Stale label May 17, 2023
@luck3y
Copy link
Contributor

luck3y commented May 31, 2023

/retest

@github-actions github-actions bot removed the Stale label Jun 1, 2023
@github-actions
Copy link

There has been no activity on this PR for 45 days. It will be auto-closed after 90 days.

@github-actions github-actions bot added the Stale label Jul 16, 2023
@github-actions
Copy link

There has been no activity on this PR for 90 days and it has been closed automatically.

@github-actions github-actions bot closed this Oct 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hold Do not merge this PR Stale
Projects
None yet
4 participants