-
Notifications
You must be signed in to change notification settings - Fork 13
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
Feature/update release flow #54
Conversation
- name: Upload image description | ||
uses: actions/upload-artifact@v2-preview | ||
with: | ||
path: desc/output/*.txt |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You changed the extension to .md
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, that is the intention of c0bb788. As those files are in MD format, and are now commited on release, this will allow to display those files through github markdown renderer.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The generated files extension was changed but not the files uploaded.
Are there files which match desc/output/*.txt
to upload ? If not, this should be replaced with desc/output/*.md or removed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I forgot to remove this step. Now image descriptions will also be versioned so no need to attach them with release artifact.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be good
5cfd55c
to
07418ad
Compare
Fix #51