-
Notifications
You must be signed in to change notification settings - Fork 153
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
CI: Expose detected plugin name to other workflow jobs #109
Conversation
e70c9f2
to
b4b38a8
Compare
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.
Seems fine.
With the new |
Right, it's not supposed to be used for file names now that it can have spaces and other weird characters in it, forgot to remove that again. |
b4b38a8
to
fe65ca4
Compare
@CodeYan01 pls check again. |
In the Issue, you commented that it will fallback to With the Thank you! |
Ah, good old |
fe65ca4
to
46eb614
Compare
Tested both cases, works now. |
Description
Adds early detection of the configured plugin name in workflows so the name can be re-used in later workflows without the need to do a full repository checkout or installation of 3rd party utilities (e.g. the release creation step).
Motivation and Context
Reduce necessary fixup steps for plugin developers when finalising their releases.
Fixes #105.
How Has This Been Tested?
Tested with a tagged release on a fork of the repository.
Types of changes
Checklist: