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

Add sortpom-maven-plugin #370

Merged
merged 2 commits into from
Sep 28, 2023
Merged

Add sortpom-maven-plugin #370

merged 2 commits into from
Sep 28, 2023

Conversation

wendigo
Copy link
Contributor

@wendigo wendigo commented Jun 27, 2023

No description provided.

wendigo added a commit to wendigo/airlift that referenced this pull request Jun 27, 2023
The plugin will be applied from the airbase 144: airlift/airbase#370
wendigo added a commit to wendigo/airlift that referenced this pull request Jun 27, 2023
The plugin will be applied from the airbase 144: airlift/airbase#370
@wendigo
Copy link
Contributor Author

wendigo commented Jun 28, 2023

Cc @electrum

@electrum
Copy link
Member

Can you remove the section headers for the dependencies? They don't always make sense now, as things are no longer logically grouped together.

@wendigo
Copy link
Contributor Author

wendigo commented Sep 19, 2023

PTAL @electrum

@wendigo
Copy link
Contributor Author

wendigo commented Sep 27, 2023

Rebased.

@electrum electrum merged commit 83486d3 into airlift:master Sep 28, 2023
2 checks passed
@wendigo wendigo deleted the serafin/sortpom branch September 28, 2023 19:50
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.

2 participants