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

bump box/spout version to 2.7.2 #5285

Closed
wants to merge 4 commits into from
Closed

bump box/spout version to 2.7.2 #5285

wants to merge 4 commits into from

Conversation

jmleroux
Copy link
Contributor

@jmleroux jmleroux commented Dec 1, 2016

This PR aims to use last 2.7.2 stable version of box/spout
It fixes some issues with blank lines for example. Usefull for the ExcelInitBundle

@jmleroux jmleroux changed the title bump box/spout version to 2.70 bump box/spout version to 2.7.1 Dec 1, 2016
@jmleroux jmleroux force-pushed the jml-bump-spout branch 2 times, most recently from bb559d1 to 10b19b8 Compare December 2, 2016 08:04
Copy link
Contributor

@skeleton skeleton left a comment

Choose a reason for hiding this comment

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

I cannot give you a GTM without the result of perfbench to check there is no regression with 1.6

@arnolanglade
Copy link
Contributor

@jmleroux What is the status this PR ?

@jmleroux
Copy link
Contributor Author

@Arn0d I had no time to run perfbench, so it's kind of suspended.

@a2xchip
Copy link
Contributor

a2xchip commented May 30, 2017

:D :D :D What exactly to review? Build status?

CHANGELOG-1.7.md Outdated
@@ -6,6 +6,9 @@
- PIM-6118: Improve attribute add select to avoid performance impact
- GITHUB-5716: Redo family mass edit form using backbonejs architecture and internal REST API

## Technical improvements
- GITHUB-5285: Update to spout 2.7.1
Copy link
Contributor

Choose a reason for hiding this comment

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

upgrade?

Copy link
Contributor

@a2xchip a2xchip May 30, 2017

Choose a reason for hiding this comment

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

To add empty line after ## Technical improvements

Copy link
Contributor

@a2xchip a2xchip left a comment

Choose a reason for hiding this comment

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

Just little comment

@jmleroux jmleroux changed the title bump box/spout version to 2.7.1 bump box/spout version to 2.7.2 May 30, 2017
Copy link
Contributor

@a2xchip a2xchip left a comment

Choose a reason for hiding this comment

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

GTM when green ))

@@ -12,6 +12,7 @@
- TIP-725: Generalization of the refactoring made in the TIP-724 for all screen containing a simple grid
- TIP-734: Menu and index page is now using the new PEF architecture
- GITHUB-6174: Show a loading mask during the file upload in the import jobs
- GITHUB-5285: Upgrade to spout 2.7.2
Copy link
Contributor

Choose a reason for hiding this comment

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

👍

@arnolanglade
Copy link
Contributor

I close this PR because it was done in 35053fb

@arnolanglade arnolanglade deleted the jml-bump-spout branch August 29, 2017 17:24
@a2xchip
Copy link
Contributor

a2xchip commented Sep 2, 2017

👍

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.

5 participants