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

JCenter to mavenCentral #4

Merged
merged 2 commits into from
Nov 2, 2022
Merged

Conversation

BetterB0y
Copy link

currently, the jcenter is down, and it's better if we can fix this on open_filex.
thank you

@BetterB0y
Copy link
Author

@javaherisaber can you look into it?

@javaherisaber
Copy link
Owner

javaherisaber commented Nov 1, 2022

@BetterB0y
According to this document, JCenter will be read-only indefinitely

JFrog has decided to keep JCenter as a read-only repository indefinitely. New package and versions are no longer accepted on JCenter. All Bintray services have been shutdown.

Have you seen any issues while building the library?

@BetterB0y
Copy link
Author

Yes. Yesterday the whole repo was 403 error and unreachable.
Here is stack thread
https://stackoverflow.com/questions/74258160/is-jcenter-down-permanently-31-oct

@BetterB0y
Copy link
Author

BetterB0y commented Nov 1, 2022

Today is also 403. Check by yourself @javaherisaber .

https://jcenter.bintray.com

@javaherisaber
Copy link
Owner

@BetterB0y
I just checked again and it's not throwing 403 for me, (I also cleaned my cache and retried)
Anyway I checked, replacing JCenter with MavenCentral satisfies the dependencies
So I can merge your change and make it future proof

Thanks for noticing and the PR

@javaherisaber javaherisaber merged commit 4a7daa7 into javaherisaber:master Nov 2, 2022
ThomasTebbe added a commit to 123erfasst/open_filex that referenced this pull request Nov 8, 2022
This reverts commit 4a7daa7, reversing
changes made to 633f896.
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.

2 participants