GithubHelp home page GithubHelp logo

Comments (10)

stefanbuck avatar stefanbuck commented on June 11, 2024 2

I'm currently working on a GitHub app which might be interesting for you @josh803316. It automatically adds a badge to the Pull Requests description that links to the related JIRA ticket.

The app is in private beta right now. Please drop me a DM if you want to join the beta group

https://twitter.com/buckstefan/status/1147620338584473602

from semantic-pull-requests.

zeke avatar zeke commented on June 11, 2024

I would prefer to keep this bot as agnostic as possible about commit conventions, deferring to an existing parser (currently parse-commit-message) that implements the Conventional Commits spec.

That said, maybe there's a case for trying to add something like this to the spec. Do you know of tools other than Jira that use this style?

from semantic-pull-requests.

josh803316 avatar josh803316 commented on June 11, 2024

@zeke Thanks for the response! Some people actually use the following format for issues that have been opened on Github repos:

[FIXES-123] or [FIX-123] ... and some do it without the brackets. It would be cool to simply add an optional prefix, although I totally understand your desire to keep the bot as agnostic as possible. Worse case I guess we could try to open a PR, if we knew where to look.

from semantic-pull-requests.

nesl247 avatar nesl247 commented on June 11, 2024

I was actually about to open up an issue when I saw this. Having support for this exact thing would be really beneficial.

One other exception that would be beneficial (if not already supported) is to not fail on would be a WIP, WIP:, [WIP] prefix as is used with many WIP bots.

from semantic-pull-requests.

damusnet avatar damusnet commented on June 11, 2024

Same thing here, a regular commit / PR will look like this where I work:
WR-13: fix(Sidebar): update font-family and letter-spacing
Adding the WR-13 allows us to link Jira tickets with GitHub PR and track them to completion.

IIUC semantic-pull-requests uses parse-commit-message which doesn't validate against such a prefix by default. It does however seem to support plugins to extend it. Not sure how it would be possible to pass a config from a repo to semantic-pull-requests to parse-commit-message though 🤔

from semantic-pull-requests.

markcellus avatar markcellus commented on June 11, 2024

I would prefer to keep this bot as agnostic as possible about commit conventions, deferring to an existing parser (currently parse-commit-message) that implements the Conventional Commits spec.

@zeke but it appears the tool isn't actually agnostic to the commit conventions though. It actually is opinionated about the type of commit conventions because it requires the style of conventional commits. :) Is there a way you would recommend to have a custom validation git commit messages that are not conventional commits?

from semantic-pull-requests.

tunnckoCore avatar tunnckoCore commented on June 11, 2024

@damusnet, parse-commit-message supports plugins (at least since v3, not sure what was the version when you commented), but I believe, it was always supporting plugins. Anyway.

@zeke could allow extending it through the semantic-pull-requests config. Or passing different regex to the parser. I'l think of something after the v4 release.

Overall, yea, semantic-pull-requests is meant to be following the conventional commits (i guess all this came from Angular and Semantic Release) and that's why it's called that way. But yea, it's not a problem to extend it. I'm open for the parse-commit-message too, since it's name is even more general.

I'm trying to find from where I published latest v4 canary (.20) version, dang... 🤕 I believe I published it from somewhere in speed to fix this bot :D Aaaaaaaanyway.

from semantic-pull-requests.

tunnckoCore avatar tunnckoCore commented on June 11, 2024

some wanting this can try implement it with latest canary (canary-20200142.1) of the parser. hint: pass options.headerRegex

from semantic-pull-requests.

tunnckoCore avatar tunnckoCore commented on June 11, 2024

4.0.0-canary.20200124.2 (parse-commit-message@canary) has full-blown documentation ;p Only more tests left.

from semantic-pull-requests.

zeke avatar zeke commented on June 11, 2024

This issue is old and inactive so I'm going to close it. If anyone is still affected by this, let me know and I'll re-open it.

from semantic-pull-requests.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.