Skip to content

Conversation

@adityamparikh
Copy link
Contributor

Apply spotless plugin for consistent code formatting

@adityamparikh
Copy link
Contributor Author

@chatman @epugh what process do we follow for subsequent PR reviews?

Copy link
Contributor

@chatman chatman left a comment

Choose a reason for hiding this comment

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

Great to have spotless configured. Lets merge this right away, since every other change would be based on this.

@chatman chatman merged commit 0db92d1 into apache:main Oct 28, 2025
@chatman
Copy link
Contributor

chatman commented Oct 28, 2025

@chatman @epugh what process do we follow for subsequent PR reviews?

I guess you just wait for some committer to review and merge your PRs, just like this one. More involved changes might require more time consuming reviews. In case you feel a PR is stalled, feel free to bring attention to it using the dev@ list of Solr.

@epugh
Copy link
Contributor

epugh commented Oct 28, 2025

i am also trying to figure out the process a bit...! Glad this is in.

epugh added a commit that referenced this pull request Oct 31, 2025
* Add Docker support with Jib and GitHub Actions CI/CD

* Minimal .asf.yaml to get correct github notifications in place

* Apply spotless plugin (#5)

* Add Docker support with Jib and GitHub Actions CI/CD

# Conflicts:
#	build.gradle.kts
#	gradle/libs.versions.toml

* Update the repo name pattern.

* Add Docker support with Jib and GitHub Actions CI/CD

# Conflicts:
#	build.gradle.kts
#	gradle/libs.versions.toml

* test: add Docker integration tests for MCP server under both STDIO and HTTP modes

---------

Co-authored-by: Chris Hostetter <hossman@apache.org>
Co-authored-by: Eric Pugh <epugh@opensourceconnections.com>
@adityamparikh adityamparikh deleted the spotless-formatting branch November 1, 2025 22:31
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.

3 participants