-
Notifications
You must be signed in to change notification settings - Fork 315
Conversation
Oh, this is handy 😍 |
dist/index.js
Outdated
core.setFailed(error.message); | ||
} | ||
} | ||
|
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.
What happened here? Spaces to tabs or vice-versa?
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.
dist/index.js
Is this file auto-generated by build? @monkeyWie Can you help fix it?
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 I revertdist / index.js
and build it by author self?
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.
@IAmHughes Help to merge this pr, pls.
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.
@deancn dist/index.js
is created and added to the commit by npm run precommit
, which is run by a pre-commit Git hook added by husky when you npm install
the project. You should not edit it yourself, but instead fix errors in src/
.
@jnovack @IAmHughes I have resolved the conflict,please review again. |
Bumps [handlebars](https://github.com/wycats/handlebars.js) from 4.2.1 to 4.5.3. - [Release notes](https://github.com/wycats/handlebars.js/releases) - [Changelog](https://github.com/wycats/handlebars.js/blob/master/release-notes.md) - [Commits](handlebars-lang/handlebars.js@v4.2.1...v4.5.3) Signed-off-by: dependabot[bot] <[email protected]>
release body based on file
…ersioning Add workflow to automate versioning
Any update on this? Would be most useful to only have the tag_name and not the /ref/tags/tag_name |
The `create-release` action strips out the `refs/tags/` from the beginning of `github.ref` and does not provide the tag name as an output (yet see actions/create-release#10).
...otherwise `body` usage sample doesn't work, which results in a bunch of open issues on the topic.
Update README to show usage of "latest" version
Bumps [acorn](https://github.com/acornjs/acorn) from 5.7.3 to 5.7.4. - [Release notes](https://github.com/acornjs/acorn/releases) - [Commits](acornjs/acorn@5.7.3...5.7.4) Signed-off-by: dependabot[bot] <[email protected]>
…ndlebars-4.5.3 Bump handlebars from 4.2.1 to 4.5.3
…orn-5.7.4 Bump acorn from 5.7.3 to 5.7.4
Update readme example to pin to major version
Bump actions/github to 2.2.0
body_path missing from action.yml
Changed release name to be optional field
…commitish Tag and release on a specific commitish
Add CodeQL security scanning
create release with body text sourced from file
Bumps [lodash](https://github.com/lodash/lodash) from 4.17.15 to 4.17.19. - [Release notes](https://github.com/lodash/lodash/releases) - [Commits](lodash/lodash@4.17.15...4.17.19) Signed-off-by: dependabot[bot] <[email protected]>
…dash-4.17.19 Bump lodash from 4.17.15 to 4.17.19
Hi folks, Could you please check on this PR, please? Kind regards, |
Won't review until all conflicts are resolved. |
Reopen a PR:#97 |
Hello,as you said,parsing the
refs/tags/xxx
is a matter in workflow,so I re open this PR.