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-6332: clean up license config files #5707

Merged
merged 2 commits into from
Nov 15, 2023
Merged

Conversation

michpetrov
Copy link
Contributor

@michpetrov michpetrov commented Oct 10, 2023

DRAFT PR pending new release of the license plugin

Issue: WFCORE-6332

@wildfly-ci

This comment was marked as off-topic.

@wildfly-ci

This comment was marked as off-topic.

@wildfly-ci

This comment was marked as outdated.

@github-actions github-actions bot added the deps-ok Dependencies have been checked, and there are no significant changes label Oct 10, 2023
@wildfly-ci

This comment was marked as outdated.

@wildfly-ci

This comment was marked as outdated.

@wildfly-ci

This comment was marked as outdated.

@michpetrov michpetrov marked this pull request as ready for review November 8, 2023 17:19
@michpetrov michpetrov requested a review from yersan November 10, 2023 13:21
@yersan
Copy link
Collaborator

yersan commented Nov 13, 2023

@michpetrov Could you please also address Brian's comment done in https://issues.redhat.com/browse/WFLY-18059? jboss-vfs is a dependency managed by WildFly Core and has not been resolved yet. Addressing that will help with the review since there are a lot of pieces involved here. Thanks.

@michpetrov
Copy link
Contributor Author

@yersan I have filed https://issues.redhat.com/browse/JBVFS-220, but it is not necessary for this issue. When jboss-vfs gets upgraded the record can be deleted.

Copy link
Collaborator

@yersan yersan left a comment

Choose a reason for hiding this comment

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

Thanks @michpetrov

When jboss-vfs gets upgraded the record can be deleted.

Could you create a WFCORE Jira blocked by JBVFS-220, to track it so we do not forget to complete this work?

@yersan yersan added the ready-for-merge This PR is ready to be merged and fulfills all requirements label Nov 15, 2023
@yersan yersan merged commit e04e5f0 into wildfly:main Nov 15, 2023
1 check passed
@michpetrov michpetrov deleted the wfcore-6332 branch November 15, 2023 14:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deps-ok Dependencies have been checked, and there are no significant changes ready-for-merge This PR is ready to be merged and fulfills all requirements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants