Skip to content

prevent duplicate assembly steps - #50

Merged
rmehner merged 1 commit into
transloadit:masterfrom
ifedapoolarewaju:prevent_duplicate_steps
May 30, 2018
Merged

prevent duplicate assembly steps#50
rmehner merged 1 commit into
transloadit:masterfrom
ifedapoolarewaju:prevent_duplicate_steps

Conversation

@ifedapoolarewaju

@ifedapoolarewaju ifedapoolarewaju commented May 20, 2018

Copy link
Copy Markdown
Contributor

fixes #27

@rmehner

rmehner commented May 21, 2018

Copy link
Copy Markdown
Collaborator

👍 This will be a breaking change tho, so we should have a CHANGELOG entry for this :)

@kvz

kvz commented May 22, 2018

Copy link
Copy Markdown
Member

+1 on CHANGELOG update, and I wonder if we want to bump the version to next major then also?

@rmehner

rmehner commented May 22, 2018

Copy link
Copy Markdown
Collaborator

bump the version to next major then also

Yup, it's a breaking change :)

@ifedapoolarewaju
ifedapoolarewaju force-pushed the prevent_duplicate_steps branch from 8d48f04 to 28878ff Compare May 22, 2018 13:08
@ifedapoolarewaju

Copy link
Copy Markdown
Contributor Author

@rmehner @kvz updated, thank you

Comment thread CHANGELOG.md Outdated
@@ -1,3 +1,7 @@
### 2.1.0 / ????-??-?? ###

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

3.0.0, since this is a breaking change.

@ifedapoolarewaju
ifedapoolarewaju force-pushed the prevent_duplicate_steps branch from 28878ff to c57ab75 Compare May 22, 2018 13:12
@ifedapoolarewaju

Copy link
Copy Markdown
Contributor Author

@rmehner done, thank you

@rmehner

rmehner commented May 22, 2018

Copy link
Copy Markdown
Collaborator

Thank you @ifedapoolarewaju. I can push out the release later today :)

@ifedapoolarewaju

Copy link
Copy Markdown
Contributor Author

@rmehner to merge or not to merge?

@rmehner
rmehner merged commit 525da8f into transloadit:master May 30, 2018
@rmehner

rmehner commented May 30, 2018

Copy link
Copy Markdown
Collaborator

Totally forgot about this. Will put this into my calendar to push out the release later today.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Detect duplicate step names

3 participants