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.

Sign up for GitHub

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

Jump to bottom

Update to ASM 9.6 + POM and code refresh #28

Merged
merged 8 commits into from
Oct 10, 2023
Merged

Update to ASM 9.6 + POM and code refresh #28

merged 8 commits into from
Oct 10, 2023

Conversation

rfelcman
Copy link
Contributor

There is output from org.eclipse.dash:license-tool-plugin

maven/mavencentral/org.ow2.asm/asm-commons/9.6, BSD-3-Clause, approved, #10775
maven/mavencentral/org.ow2.asm/asm-tree/9.6, BSD-3-Clause, approved, #10773
maven/mavencentral/org.ow2.asm/asm/9.6, BSD-3-Clause, approved, #10776

@rfelcman
Copy link
Contributor Author

JDK 21 test fails due eclipse.dash tool. Local build works.

@lukasj
Copy link
Member

lukasj commented Oct 10, 2023

dash invocation can be moved to an extra step (and it will run just once..)

@rfelcman
GH actions change
39d76a6 
Signed-off-by: Radek Felcman <[email protected]>
@rfelcman
GH actions change
95623a6 
Signed-off-by: Radek Felcman <[email protected]>
@rfelcman
GH actions change
eb58df6 
Signed-off-by: Radek Felcman <[email protected]>
@rfelcman
GH actions change
d7e796d 
Signed-off-by: Radek Felcman <[email protected]>
@rfelcman
Copy link
Contributor Author

dash invocation can be moved to an extra step (and it will run just once..)

It works now.

@lukasj lukasj merged commit 2e9ce91 into eclipse-ee4j:master Oct 10, 2023
3 checks passed
@rfelcman rfelcman deleted the asm_update_96 branch October 10, 2023 12:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lukasj lukasj lukasj left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@rfelcman @lukasj