Skip to content

Conversation

@james-willis
Copy link
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings July 11, 2025 18:16
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds Java 11 compatibility by replacing Java 17 pattern matching in equals, and updates CI workflows to run releases on pushes to main and build/tests on pull requests to main.

  • Revert instanceof pattern matching in Config.equals for Java 11 support
  • Trigger .github/workflows/release.yml on push to main
  • Run build-and-test workflow on pull_request targeting main

Reviewed Changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated no comments.

File Description
src/main/java/com/mapzen/jpostal/Config.java Replace Java 17 instanceof pattern matching with classic cast
.github/workflows/release.yml Add push trigger for main branch
.github/workflows/build-and-test.yml Add pull_request trigger for main branch
Comments suppressed due to low confidence (2)

.github/workflows/release.yml:4

  • Indentation of 'push' under 'on:' should match the indentation level used for 'workflow_dispatch' to maintain consistent YAML formatting.
  push:

.github/workflows/build-and-test.yml:3

  • Ensure 'pull_request' is indented consistently with other event triggers ('push', 'workflow_call') under 'on:' for a valid YAML structure.
  pull_request:

@james-willis james-willis merged commit 2fc3cce into main Jul 11, 2025
5 checks passed
@james-willis james-willis deleted the java-11 branch July 11, 2025 18:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants