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

CXMM-5894: Updating project gradle #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

AZenhom
Copy link

@AZenhom AZenhom commented Apr 12, 2023

https://uplandsoftware.atlassian.net/browse/CXMM-5894

  • Updating project Gradle to 7+
  • Updating Project dependencies
  • Updating deprecated Gradle statements
  • Removing Maven Publishing scripts after the SDK team decided to upload to Jfrog manually

@AZenhom AZenhom requested review from JuJu227 and bdale-upland April 12, 2023 14:46
@AZenhom AZenhom self-assigned this Apr 12, 2023
@bdale-upland bdale-upland changed the title Updating project gradle CXMM-5894: Updating project gradle Apr 12, 2023
@bdale-upland
Copy link

bdale-upland commented Apr 12, 2023

Looks like there are TravisCI issues here

@bdale-upland
Copy link

bdale-upland commented Apr 17, 2023

Looks like there are TravisCI issues here :(

Spoke with Ahmed about this, TravisCI not hooked up here, not relevant at all due to how the build is setup.

Ahmed said: "Since we decided at last to manually push to Jfrog … I removed the maven-publishing scripts that i implemented earlier … The PR now only contains the Gradle and Java Updates …"

@bdale-upland
Copy link

Needs @JuJu227 review.

NOTE: no huge rush, Ahmed out till next week (Wed Apr 26) d/t Egyptian holiday, starting tomorrow (Apr 20)

@JuJu227
Copy link

JuJu227 commented Apr 19, 2023

@AZenhom Hey, could you get the Travis-ci to build and publish to jfrog? If that's outside of your comfort zone, that's fine I will wrap up that aspect later.

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

Successfully merging this pull request may close these issues.

3 participants