Skip to content

feat(echo): Add echo maven support#5555

Merged
Ly-Joey merged 6 commits into
google:masterfrom
LaizaAngrest:add-echo-maven-support
Jun 30, 2026
Merged

feat(echo): Add echo maven support#5555
Ly-Joey merged 6 commits into
google:masterfrom
LaizaAngrest:add-echo-maven-support

Conversation

@LaizaAngrest

Copy link
Copy Markdown
Contributor

Extends the Echo ecosystem to support Maven versioning semantics via the Echo:Maven suffix, allowing Echo to publish advisories for secured Maven packages without affecting the unrelated upstream Maven ecosystem. Versioning is delegated to the underlying Maven ecosystem helper, mirroring the existing Echo:PyPI handling (#5286).

Extends the Echo ecosystem to support Maven versioning semantics via the Echo:Maven suffix, allowing Echo to publish advisories for secured Maven packages without affecting the unrelated upstream Maven ecosystem. Versioning is delegated to the underlying Maven ecosystem helper, mirroring the existing Echo:PyPI handling (google#5286).
@Ly-Joey

Ly-Joey commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

cc: @orizerah since you are our usual contact at Echo.
I noticed this is a first-time contributor, so I wanted to double check if this PR adding Maven support is being driven by someone on the Echo side?

As a general heads-up, we also recently updated our CONTRIBUTING.md to ask that an issue to be created to discuss proposed changes before making a PR, just to make sure efforts are aligned with project goals and to avoid duplicate work.

@orizerah

Copy link
Copy Markdown
Contributor

@Ly-Joey

Hey,
The Maven support is driven from us and @LaizaAngrest is joining as a contributor from Echo.

We expect to contribute NPM support soon as well.

We will open issues for future contributions, do you want us to open one for this feature as well?

@orizerah

Copy link
Copy Markdown
Contributor

Hey @Ly-Joey @another-rex do you think we will be able to merge this soon?

@another-rex

Copy link
Copy Markdown
Contributor

/gcbrun

@another-rex

Copy link
Copy Markdown
Contributor

/gcbrun

@another-rex

Copy link
Copy Markdown
Contributor

Hello, the team was out this week, we will try to get this merged in next week. The code change LGTM.

fyi @Ly-Joey @cuixq

@another-rex

Copy link
Copy Markdown
Contributor

/gcbrun

@Ly-Joey

Ly-Joey commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

/gcbrun

@Ly-Joey

Ly-Joey commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

/gcbrun

@Ly-Joey Ly-Joey requested a review from cuixq June 30, 2026 00:49
@Ly-Joey Ly-Joey merged commit 4917161 into google:master Jun 30, 2026
26 checks passed
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.

5 participants