Skip to content

fix: allow overwriting existing directories#87

Merged
asyncapi-bot merged 2 commits intoasyncapi:masterfrom
XerxesDGreat:master
Feb 2, 2023
Merged

fix: allow overwriting existing directories#87
asyncapi-bot merged 2 commits intoasyncapi:masterfrom
XerxesDGreat:master

Conversation

@XerxesDGreat
Copy link
Contributor

Description
Test for existing directory structure in order to prevent failures when updating an existing application using the generator

Related issue(s)
Fixes #86

Resolves asyncapi#86. Now allows running the generator on an existing directory structure
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Welcome to AsyncAPI. Thanks a lot for creating your first pull request. Please check out our contributors guide useful for opening a pull request.
Keep in mind there are also other channels you can use to interact with AsyncAPI community. For more details check out this issue.

Copy link
Collaborator

@dalelane dalelane left a comment

Choose a reason for hiding this comment

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

LGTM - thanks for taking the time to fix this 👍

Copy link
Member

@dan-r dan-r left a comment

Choose a reason for hiding this comment

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

LGTM - thank you

@dan-r
Copy link
Member

dan-r commented Feb 2, 2023

/rtm

@asyncapi-bot asyncapi-bot merged commit 9ef6a64 into asyncapi:master Feb 2, 2023
@asyncapi-bot
Copy link
Contributor

🎉 This PR is included in version 0.2.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

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.

Running the generator on an existent file system fails

4 participants