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

Update GitHub action try fix extended tests #4166

Closed
wants to merge 24 commits into from

Conversation

vga91
Copy link
Collaborator

@vga91 vga91 commented Aug 5, 2024

Initially the error was neo4j 5.23 not found: https://github.com/neo4j-contrib/neo4j-apoc-procedures/actions/runs/10453968982/job/28945523530

Update GitHub action try fix extended tests

/home/runner/work/neo4j-apoc-procedures/neo4j-apoc-procedures/extended/src/main/java/apoc/custom/CypherProceduresHandler.java:254: error: <anonymous apoc.custom.CypherProceduresHandler$1> is not abstract and does not override abstract method apply(Context,AnyValue[],ResourceMonitor) in BasicProcedure
            globalProceduresRegistry.register(new CallableProcedure.BasicProcedure(signature) {
                                                                                              ^
/home/runner/work/neo4j-apoc-procedures/neo4j-apoc-procedures/extended/src/main/java/apoc/custom/CypherProceduresHandler.java:256: error: apply(Context,AnyValue[],ResourceMonitor) in <anonymous apoc.custom.CypherProceduresHandler$1> cannot override apply(Context,AnyValue[],ResourceMonitor) in BasicProcedure
  • Asked to Omar via Slack to solve the docker error: TODO
    Tried in this way
          curl -s -L0 -u "${CODEARTIFACT_USERNAME}:${CODEARTIFACT_TOKEN}" -X GET ${DOCKER_ENTERPRISE_URL} -o ${ENTERPRISE_TAR} &
          curl -s -L0 -u "${CODEARTIFACT_USERNAME}:${CODEARTIFACT_TOKEN}" -X GET ${DOCKER_COMMUNITY_URL} -o ${COMMUNITY_TAR} &
          wait
          docker load --input ${ENTERPRISE_TAR}
          docker load --input ${COMMUNITY_TAR}

@vga91 vga91 marked this pull request as draft August 5, 2024 12:55
@vga91 vga91 force-pushed the update-github-action-try-fix-extended-tests branch 3 times, most recently from 1d8e08f to 7590227 Compare August 6, 2024 07:52
@vga91 vga91 force-pushed the update-github-action-try-fix-extended-tests branch from ff9ae48 to e39daf7 Compare August 19, 2024 12:33
@vga91 vga91 force-pushed the update-github-action-try-fix-extended-tests branch from e39daf7 to 6dc05a2 Compare August 19, 2024 13:06
@vga91 vga91 force-pushed the update-github-action-try-fix-extended-tests branch from ef6127a to 41e7bfe Compare August 20, 2024 07:22
@vga91 vga91 force-pushed the update-github-action-try-fix-extended-tests branch from 9261f96 to 9ba4f74 Compare August 22, 2024 07:57
@vga91 vga91 force-pushed the update-github-action-try-fix-extended-tests branch from 9ba4f74 to 56b627c Compare August 22, 2024 07:59
@vga91 vga91 force-pushed the update-github-action-try-fix-extended-tests branch from ff445d8 to 35e55cb Compare August 28, 2024 08:03
@vga91
Copy link
Collaborator Author

vga91 commented Sep 4, 2024

No longer valid

@vga91 vga91 closed this Sep 4, 2024
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.

1 participant