Skip to content

Conversation

@nclsHart
Copy link
Contributor

@nclsHart nclsHart commented Mar 29, 2018

Should be rebased and merged after #18.

This is a mandatory step before upgrading swiftmailer to 6.x, which require php >= 7.0.0.

Implements #28.

@hkdobrev
Copy link
Contributor

@nclsHart Thanks! 🙌 Could you please rebase that on top of latest master as I've merged your other PRs?

@hkdobrev
Copy link
Contributor

I've added you as a collaborator to the project 🎉 Please use PRs and let's discuss releases and upgrades in issues for now :) Cheers!

@nclsHart nclsHart force-pushed the remove-unsupported-versions branch from 8589209 to 90fc93e Compare March 29, 2018 09:12
@hkdobrev hkdobrev self-requested a review March 29, 2018 09:41
@hkdobrev
Copy link
Contributor

hkdobrev commented Mar 29, 2018

BTW I'm fine with dropping PHP 5 altogether and adding PHP 7.2.

@nclsHart
Copy link
Contributor Author

@hkdobrev Thanks for the invitation :).

My idea was to have something like the following releases:

0.5

  • php versions supported: 5.4, 5.5, 5.6, 7.0 and 7.1
  • swiftmailer: 5.x
  • css-to-inline-styles: 2.1.x

This is the current state of master.

0.6

  • php versions supported: 5.6, 7.0, 7.1, 7.2
  • swiftmailer: 5.x
  • css-to-inline-styles: 2.2.x

0.7 or 1.0.0

  • php versions supported: 7.0, 7.1, 7.2
  • swiftmailer: 6.x
  • css-to-inline-styles: 2.2.x

What do you think about it? Maybe we could discuss this in a dedicated issue.

@nclsHart nclsHart force-pushed the remove-unsupported-versions branch from 90fc93e to 737c678 Compare March 29, 2018 10:09
@nclsHart nclsHart changed the title [WIP] Remove unsupported php versions Remove unsupported php versions Mar 29, 2018
@hkdobrev
Copy link
Contributor

hkdobrev commented Mar 29, 2018

OK, let's do that! However, tests are failing for me locally. Opened a separate issue for that: #21.

@nclsHart nclsHart force-pushed the remove-unsupported-versions branch from 737c678 to a7b4cd0 Compare March 29, 2018 12:24
@nclsHart nclsHart added this to the 0.6.0 milestone Apr 5, 2018
@hkdobrev
Copy link
Contributor

hkdobrev commented Apr 5, 2018

Could you please rebase on top of latest master to resolve conflicts? Thanks!

@hkdobrev hkdobrev changed the title Remove unsupported php versions Remove EOL php versions Apr 5, 2018
@nclsHart nclsHart force-pushed the remove-unsupported-versions branch from a7b4cd0 to 67e3e20 Compare April 5, 2018 14:57
@nclsHart nclsHart mentioned this pull request Apr 5, 2018
2 tasks
@nclsHart
Copy link
Contributor Author

nclsHart commented Apr 6, 2018

As discuss in #28, I'll split this in two PR.

@nclsHart nclsHart closed this Apr 6, 2018
@nclsHart nclsHart deleted the remove-unsupported-versions branch April 6, 2018 09:40
@nclsHart nclsHart removed this from the 0.6.0 milestone Apr 6, 2018
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.

2 participants