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-6870][WFCORE-6871][WFCORE-6876] ModuleSpecification cleanups #6053

Merged
merged 3 commits into from
Jul 9, 2024

Conversation

bstansberry
Copy link
Contributor

@bstansberry bstansberry commented Jun 29, 2024

@github-actions github-actions bot added the deps-ok Dependencies have been checked, and there are no significant changes label Jun 29, 2024
@wildfly-ci

This comment was marked as off-topic.

@bstansberry
Copy link
Contributor Author

Note that the GH status box only shows 5 CI jobs passing, but https://ci.wildfly.org/project.html?projectId=WildFlyCore_PullRequest&branch_WildFlyCore_PullRequest=pull%2F6053 show the usual larger set.

IIRC I forgot to add a file in the branch before submitting this and quickly fixed that, which may have confused things.

@wildfly-ci

This comment was marked as outdated.

@bstansberry
Copy link
Contributor Author

The status box doesn't show the full set of CI results. They can be seen here:

https://ci.wildfly.org/project.html?projectId=WildFlyCore_PullRequest&branch_WildFlyCore_PullRequest=pull%2F6053

All jobs pass except the one recorded in the status box (which is unrelated).

@wildfly-ci

This comment was marked as outdated.

@bstansberry bstansberry changed the title [WFCORE-6870][WFCORE-6871] ModuleSpecification cleanups [WFCORE-6870][WFCORE-6871][WFCORE-6876] ModuleSpecification cleanups Jul 3, 2024
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.

This looks safe enough to me for going to Final, it also improves the tests, let me know if you disagree

@yersan yersan added 25.x ready-for-merge This PR is ready to be merged and fulfills all requirements labels Jul 8, 2024
@bstansberry bstansberry merged commit 8d0d18f into wildfly:main Jul 9, 2024
5 checks passed
@bstansberry bstansberry deleted the WFCORE-6870 branch July 9, 2024 22:13
@bstansberry
Copy link
Contributor Author

Thanks @yersan

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