Skip to content
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

[12.0][MIG] stock_request_kanban #634

Merged
merged 16 commits into from Jul 11, 2019

Conversation

kittiu
Copy link
Member

@kittiu kittiu commented Jun 27, 2019

Done migrated

@kittiu kittiu mentioned this pull request Jun 27, 2019
54 tasks
Copy link
Member

@etobella etobella left a comment

Choose a reason for hiding this comment

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

squash translation commits please 😄

@kittiu
Copy link
Member Author

kittiu commented Jun 27, 2019

squash translation commits please

Yes, I see lots of pot commit here. Question is, do squash only what is before the meaningful one, i..e, 801bea4 or find all in the history and quash all .pot (now sure you understand my question, sorry)

1329fcd6 (HEAD -> 12.0-mig-stock_request_kanban, origin/12.0-mig-stock_request_kanban) [12.0][MIG] stock_request_kanban
3487d283 Update translation files
2b4ed27a [UPD] Update stock_request_kanban.pot
d9b5f56a Update translation files
17419cb9 [UPD] Update stock_request_kanban.pot
8587373b [UPD] README.rst
801bea44 fixup! [IMP] Stock_request_kanban: Set barcode configuration on parameters
49bdf007 [IMP] Stock_request_kanban: Set barcode configuration on parameters
2fbf0231 Update translation files
f5e3f26f [UPD] Update stock_request_kanban.pot
9eb7b333 Update translation files
180eaca5 [UPD] Update stock_request_kanban.pot
744d3f84 Update translation files
1a3f97b3 [UPD] Update stock_request_kanban.pot
ce06051e Update translation files

@rousseldenis rousseldenis added this to the 12.0 milestone Jul 3, 2019
Copy link
Sponsor Contributor

@rousseldenis rousseldenis left a comment

Choose a reason for hiding this comment

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

LGTM

@LoisRForgeFlow
Copy link
Contributor

@kittiu I think you can squash all automatics commits that contain .pot files, as long as you keep the non-automatic ones (done by developers) it is fine and we keep history less noisy.

Copy link
Contributor

@LoisRForgeFlow LoisRForgeFlow left a comment

Choose a reason for hiding this comment

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

Code review LG. I just did a rebase in order to go ahead with the merge.

@LoisRForgeFlow
Copy link
Contributor

/ocabot merge

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Rebased to 12.0-ocabot-merge-pr-634-by-lreficent-bump-no, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 93d78de into OCA:12.0 Jul 11, 2019
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at b3cf7d3. Thanks a lot for contributing to OCA. ❤️

PS: Don't worry if GitHub says there are unmerged commits: it is due to a rebase before merge. All commits of this PR have been merged into 12.0.

OCA-git-bot added a commit that referenced this pull request Jul 11, 2019
Signed-off-by lreficent
@MiquelRForgeFlow
Copy link
Contributor

Bitches! You did a merge with a red runbot! Solved in #659.

shame

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

Successfully merging this pull request may close these issues.

None yet