-
Notifications
You must be signed in to change notification settings - Fork 32
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 lightweight tags variable funcionality, resolves #14 #17
Conversation
…s to be described
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.
Awesome change, would just like a few more docs!
@@ -18,6 +18,10 @@ functions: | |||
name: ${self:provider.stage}-${self:service}-process-event-batch | |||
description: ${git:branch} - ${git:describe} - ${git:sha1} | |||
|
|||
processEventBatch2: |
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.
Since describeLight isn't a built-in for git, can you add some more documentation to the README as to what it does?
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.
Sure thing :)
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.
Commit #44a193e
has added documentation as requested.
Added explanation of `describeLight` and its difference to `describe`
@domroutley thanks so much for the improvement! |
@domroutley Released as 3.2.0 |
This Pull Request adds the ability for users to use a variable in serverless
${git:describeLight}
that will resolve to
git describe --always --tags
. This is desirable because it will allow the display of lightweight tags (such as those generated when using GitHub releases). resolves #14Checklist