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 file output parameters to Maven plugin #502

Merged
merged 10 commits into from
Jul 25, 2023

Conversation

studur
Copy link
Contributor

@studur studur commented Mar 28, 2023

No tests added yet.
Will change the file handling to prevent overwritting existing files (like system files :) ). This handling will be extracted in its own class.
Will add filename string sanitization and related tests.

Closes #501

@studur
Copy link
Contributor Author

studur commented Mar 29, 2023

@joschi thank you for all the work you put into this. We would need the the same file output functionnality as in the CLI but in the maven plugin as proposed in #501. However, we are working woth a private bitbucket and a fork is not the best solution for us since we want to be able to sync with the latest. That is why I propose the changes, hoping that these will be available publicly and allow us to use the original plugin in our code. Let me know if you have any suggestions or comments. Thanx

@studur studur marked this pull request as ready for review March 31, 2023 01:31
@joschi joschi changed the title issue-501: Added fileoutput parameters on maven plugin. Add file output parameters to Maven plugin Jul 25, 2023
@joschi joschi added this to the 2.1.0 milestone Jul 25, 2023
joschi added 3 commits July 25, 2023 22:50
Conflicts:
	maven/src/main/java/org/openapitools/openapidiff/maven/OpenApiDiffMojo.java
Copy link
Contributor

@joschi joschi left a comment

Choose a reason for hiding this comment

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

@studur Thanks for your contribution!

@joschi joschi merged commit 2bd7351 into OpenAPITools:master Jul 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add output to files in maven plugin
2 participants