Skip to content

Conversation

@tpluscode
Copy link
Contributor

fixes #7

@AlexZeitler AlexZeitler merged commit 6076792 into PDMLab:master Oct 28, 2019
@AlexZeitler
Copy link
Contributor

Thanks, 📦 express-http-problem-details@0.1.2 has been released.

@tpluscode tpluscode deleted the 4-param-express-handler branch October 28, 2019 08:08
@AlexZeitler
Copy link
Contributor

I'm just wondering if the release number should have been 0.2.0 after merging and releasing this PR?

At least thats how I understood the FAQ:

fix type commits should be translated to PATCH releases. feat type commits should be translated to MINOR releases. Commits with BREAKING CHANGE in the commits, regardless of type, should be translated to MAJOR releases.

Did I do something wrong?

@tpluscode
Copy link
Contributor Author

Indeed it should have. I did not pay attention to that.

Did you pass any parameters to standard-version?

@AlexZeitler
Copy link
Contributor

No, I just ran yarn release

@AlexZeitler
Copy link
Contributor

Same happened for the other libs where we added standard-version

@tpluscode
Copy link
Contributor Author

tpluscode commented Nov 4, 2019

That is strange indeed. I've been using these tools for a few months now without such issues.

One thing though. Have you been squashing these PRs?

Doesn't look like it...

@AlexZeitler
Copy link
Contributor

No I didn't squash them. So I should squash and use a conventional commit style message for it?

@tpluscode
Copy link
Contributor Author

That is a choice. I asked because squashing essentially discard the original commits from history. So the feat: could get lost and thus distorting the result of standard-version.

@AlexZeitler
Copy link
Contributor

Yes, that's why I did not squash until now. If I squash I would have to find the correct type of conventional commit type myself which imho would render usage of a tool like conventional commit / standard-version sort of less useful.

@AlexZeitler
Copy link
Contributor

Maybe I didn't read the manuals correctly but I didn't find something like "this is your workflow now"

@tpluscode
Copy link
Contributor Author

If I squash I would have to find the correct type of conventional commit type myself which imho would render usage of a tool like conventional commit / standard-version sort of less useful.

Yes. I stopped squashing since I use conventional commits. And I pay much more attention to meaningful and cohesive commits. Before it was a mess.

Maybe I didn't read the manuals correctly but I didn't find something like "this is your workflow now"

I don't follow

@AlexZeitler
Copy link
Contributor

If I squash I would have to find the correct type of conventional commit type myself which imho would render usage of a tool like conventional commit / standard-version sort of less useful.

Yes. I stopped squashing since I use conventional commits. And I pay much more attention to meaningful and cohesive commits. Before it was a mess.

So I should not have done it wrong here?

Maybe I didn't read the manuals correctly but I didn't find something like "this is your workflow now"

I don't follow

I mean: An example where the workflow using cc/sv is shown end to end.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants