Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

SS4 Upgrade. #24

Merged
merged 1 commit into from
Feb 28, 2018
Merged

SS4 Upgrade. #24

merged 1 commit into from
Feb 28, 2018

Conversation

bummzack
Copy link
Contributor

Updated readme.
Moved to TractorCow\OpenGraph namespace.
Adhere/enforce SilverStripe coding conventions.
Added a few unit-tests (not much, but better than nothing).
Added .editorconfig, .upgrade.yml.
Added phpunit, phpcs, scrutinizer and travis configs.

@tractorcow
Copy link
Owner

Hi @bummzack I've merged up 3.1 into master and branched off 3, so that you can merge this into master (we'll alias as 4). can you rebase?

@bummzack
Copy link
Contributor Author

@tractorcow Should be good to merge now. I've added the following branch alias: "dev-master": "4.x-dev"

@bummzack
Copy link
Contributor Author

If you're deciding to add travis: I just saw that tests probably won't pass with the current composer.json, since they use SiteTree… so I guess you'd have to add cms as a dev-dependency… (probably better than adding it as a requirement, since this module could also be used without cms)

Updated readme.
Moved to TractorCow\OpenGraph namespace.
Adhere/enforce SilverStripe coding conventions.
Added a few unit-tests (not much, but better than nothing).
Added .editorconfig, .upgrade.yml.
Added phpunit, phpcs, scrutinizer and travis configs.
@bummzack
Copy link
Contributor Author

I've fixed the composer.json to include cms as dev-dependency. Travis should now work if you choose to enable it.

I also fixed the PR by removing the code folder that was merged back in 😅

@tractorcow
Copy link
Owner

Oops sorry :)

@tractorcow tractorcow merged commit 8f0c9e0 into tractorcow:master Feb 28, 2018
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants