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

Failed Build #951

Closed
Kodak1234 opened this issue Jan 21, 2022 · 10 comments
Closed

Failed Build #951

Kodak1234 opened this issue Jan 21, 2022 · 10 comments

Comments

@Kodak1234
Copy link

The latest release build is failing, check jitpack.io. So unable to import project into android studio

@Kodak1234 Kodak1234 changed the title Failed Builded Failed Build Jan 22, 2022
@Kodak1234
Copy link
Author

image

@hypest
Copy link
Contributor

hypest commented Jan 24, 2022

Thanks for reporting!

Here's the build log on the JitPack job:

* What went wrong:
Execution failed for task ':picasso-loader:prepareToPublishToS3'.
> Either tagName or both branch name and sha1 command line arguments needs to be provided!
  Example usages:
  --tag-name={tag-name}
  --pull-request-number={pull-request-number} --branch-name={branch-name} --sha1={sha1-commit-hash}
  --branch-name={branch-name} --sha1={sha1-commit-hash}

👋 @oguzkocer , would you have the chance to have a look? Looks like a failure that happens on multiple (older) releases, not sure if this is due to some recent change or not. Thanks!

@hypest
Copy link
Contributor

hypest commented Feb 2, 2022

My ping above probably didn't go through (it was erroneously left inside the code block) so, let me add this new comment with it. 👋 @oguzkocer , can you have a look to this ticket? Thanks!

@oguzkocer
Copy link
Contributor

@hypest You're correct, I didn't receive the previous ping. We no longer use Jitpack as we started publishing our artifacts to our S3 Maven through a Buildkite job instead. I created a PR to update the integration instructions in README: #954

@Kodak1234 If you add the following maven url to your repositories, you should be able to fetch the latest artifacts:

repositories {
    maven { url "https://a8c-libs.s3.amazonaws.com/android" }
}

Please feel free to ping me if you have any issues.

@Kodak1234
Copy link
Author

Kodak1234 commented Feb 3, 2022

I'm still unable to pull the latest artifacts.

allprojects {
    repositories {
        google()
        jcenter()
        maven { url "https://a8c-libs.s3.amazonaws.com/android" }
        maven { url 'https://jitpack.io' }
    }
}

@oguzkocer
Copy link
Contributor

oguzkocer commented Feb 3, 2022

@Kodak1234 Which version are you trying to fetch? Please note that only the latest few versions are available in the S3 maven repository, v1.5.4 being one of them.

If you can share the line where you're adding the dependency, that'd be helpful.

Edit: The issue was most likely because of the change in artifact path which I've commented on separately below. Just making this edit, so it points to the correct comment.

@oguzkocer
Copy link
Contributor

@Kodak1234 I forgot that we changed the path of the artifact. Can you update your dependency as below and try again?

dependencies {
    api "org.wordpress:aztec:v1.5.4"
}

Sorry about that confusion!

@Kodak1234
Copy link
Author

@oguzkocer Please can you guys write a documentation on the best way to implement this editor in recyclerview, using a simple textview does not render all the spans properly.

@oguzkocer
Copy link
Contributor

@Kodak1234 I unfortunately don't have experience working with the library, but I'll bring it up internally to see if anyone who has experience have the bandwidth to write such a documentation.

In the meantime, your best best might be to look into how it's used in other WordPress projects. Here is the module it's used in the Gutenberg project and here is the one for WordPress-Android. In WordPress-Android, it might be used in other modules as well, so you might want to do a full search in the project.

I'll follow up about the internal discussion next week. Hope that helps!

@hypest
Copy link
Contributor

hypest commented Feb 4, 2022

Please can you guys write a documentation on the best way to implement this editor in recyclerview

👋 @Kodak1234, there doesn't seem to be available bandwidth from the main maintainers at the moment to dive into this request, sorry 😞 . That said, the links @oguzkocer shared above can be a good resource to check.

using a simple textview does not render all the spans properly.

I'd suggest to open a new ticket to focus on the specific usecase, ideally including links to code that can be reviewed. Aztec has been designed to be a replacement to an EditText, and shouldn't need any special handling to make it work in a RecyclerView. I could be forgetting something though so yeah, best if, in a separate ticket, you can provide a code example that fails to render the spans. Thanks!

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

No branches or pull requests

3 participants