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.

Sign up for GitHub

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

Jump to bottom

[3.x] Modernisation #40

Merged
merged 8 commits into from
May 19, 2021
Merged

[3.x] Modernisation #40

merged 8 commits into from
May 19, 2021

Conversation

clemblanco
Copy link
Contributor

@clemblanco clemblanco commented May 18, 2021 •
edited
Loading

  • No functionality change
  • Linting
  • Minor modernisations
  • Moved from Travis CI to Github Actions

clemblanco and others added 4 commits May 18, 2021 11:25
@clemblanco
WIP
5b70110
@clemblanco
WIP
fdffaa2
@clemblanco
Apply fixes from StyleCI (#41)
cf2e616 
Co-authored-by: clemblanco <clemblanco@users.noreply.github.com>
@clemblanco
WIP
8bfc72e
willthorpe
willthorpe approved these changes May 18, 2021
View reviewed changes
Copy link
Contributor

@willthorpe willthorpe left a comment

Choose a reason for hiding this comment

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

Just a couple of minor grammar things.

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
composer.json Show resolved Hide resolved
.github/workflows/run-tests.yml Show resolved Hide resolved
@clemblanco clemblanco changed the title v3.x - Modernisation [3.x] Modernisation May 19, 2021
clemblanco and others added 3 commits May 19, 2021 16:57
@clemblanco
WIP
e67722e
@clemblanco
WIP
c0ee557
@clemblanco
Apply fixes from StyleCI (#47)
9e88bae 
[ci skip] [skip ci]

Co-authored-by: clemblanco <clemblanco@users.noreply.github.com>
@clemblanco clemblanco merged commit d4bdb6f into v3.x May 19, 2021
@clemblanco clemblanco deleted the feature/v3.x/modernisation branch May 19, 2021 16:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@RichToms RichToms RichToms approved these changes

@willthorpe willthorpe willthorpe approved these changes

@francescopepe francescopepe Awaiting requested review from francescopepe

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@clemblanco @RichToms @willthorpe