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

added jakarta namespace #482

Merged
merged 1 commit into from
Jul 17, 2024
Merged

added jakarta namespace #482

merged 1 commit into from
Jul 17, 2024

Conversation

PeJetuz
Copy link

@PeJetuz PeJetuz commented Jul 15, 2024

@yegor256
Hi!
Is it ok? I want to add the jakarta namespace in takes 1307
Or these dependencies should be in takes only?

@yegor256
Copy link
Member

@PeJetuz thanks for your contribution! It's good to have these deps in jcabi-parent. However, we can't merge now because some CI actions are not passing. Please, fix them.

@PeJetuz PeJetuz force-pushed the jakarta_namespace branch from 24a658e to 08edaf7 Compare July 16, 2024 14:54
@PeJetuz
Copy link
Author

PeJetuz commented Jul 16, 2024

@yegor256 fixed. Thanks!

@yegor256
Copy link
Member

@PeJetuz not really :(

@PeJetuz PeJetuz force-pushed the jakarta_namespace branch from 08edaf7 to 121aaff Compare July 16, 2024 15:30
@PeJetuz
Copy link
Author

PeJetuz commented Jul 16, 2024

I think I fixed it, but I'm not sure :(
I'm sorry, can I check the build without disturbing you?

@PeJetuz PeJetuz force-pushed the jakarta_namespace branch from 121aaff to 0d3b6dc Compare July 16, 2024 16:02
@yegor256
Copy link
Member

@PeJetuz see the output of XCOP. Empty lines are prohibited in XML.

@PeJetuz PeJetuz force-pushed the jakarta_namespace branch from 0d3b6dc to e6ac430 Compare July 16, 2024 18:44
@PeJetuz
Copy link
Author

PeJetuz commented Jul 16, 2024

@yegor256
Thank you! I installed Ruby with XCOP and fixed the pom.xml.
I found a red '\n' in the output, it was hard to see in the large output :(
I hope the checks are now successful.

@yegor256
Copy link
Member

@rultor merge

@rultor
Copy link
Contributor

rultor commented Jul 17, 2024

@rultor merge

@yegor256 OK, I'll try to merge now. You can check the progress of the merge here

@rultor rultor merged commit 2c294da into jcabi:master Jul 17, 2024
8 checks passed
@rultor
Copy link
Contributor

rultor commented Jul 17, 2024

@rultor merge

@yegor256 Done! FYI, the full log is here (took me 7min)

@yegor256
Copy link
Member

@rultor release, tag is 0.68.0

@rultor
Copy link
Contributor

rultor commented Jul 17, 2024

@rultor release, tag is 0.68.0

@yegor256 OK, I will release it now. Please check the progress here

@rultor
Copy link
Contributor

rultor commented Jul 17, 2024

@rultor release, tag is 0.68.0

@yegor256 Done! FYI, the full log is here (took me 12min)

@yegor256
Copy link
Member

@PeJetuz thanks a lot, new version (0.68.0) released to Maven Central

@PeJetuz PeJetuz deleted the jakarta_namespace branch July 17, 2024 09:23
@PeJetuz
Copy link
Author

PeJetuz commented Jul 17, 2024

@yegor256 Thank you very much!
Please review if possible 1309.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants